After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 312665 - esddsp doesn't intercept open64, fopen, fopen64
esddsp doesn't intercept open64, fopen, fopen64
Status: RESOLVED FIXED
Product: esound
Classification: Deprecated
Component: general
0.2.31
Other All
: High normal
: ---
Assigned To: Esound Maintainers
Esound Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-08-05 11:20 UTC by Peter Åstrand
Modified: 2006-08-08 17:45 UTC
See Also:
GNOME target: ---
GNOME version: Unversioned Enhancement


Attachments
Patch against CVS, to intercept open64/fopen/fopen64 (3.71 KB, patch)
2005-08-05 11:21 UTC, Peter Åstrand
none Details | Review

Description Peter Åstrand 2005-08-05 11:20:40 UTC
esddsp doesn't intercept open64, fopen, fopen64. This makes esddsp fail with
applications like wine and sox.
Comment 1 Peter Åstrand 2005-08-05 11:21:43 UTC
Created attachment 50269 [details] [review]
Patch against CVS, to intercept open64/fopen/fopen64
Comment 2 Brent Smith (smitten) 2005-08-06 00:10:02 UTC
Marking as new, and setting priority high because the patch.
Comment 3 David Schleef 2005-08-13 21:31:24 UTC
What needs to be done to make sure this works on non-glibc systems?
Comment 4 Peter Åstrand 2005-08-14 19:38:51 UTC
>What needs to be done to make sure this works on non-glibc systems?

Are you referring to the issue of this bug, or the principle with esddsp in
general? The principle with esddsp/LD_PRELOAD should work on Solaris as well.
I'm using it with Xalf, and it works fairly well. 
Comment 5 David Schleef 2005-08-14 23:09:46 UTC
First of all, I don't want to release a version of esound that doesn't, say,
compile on Solaris or FreeBSD.  Second, I want to make sure that this patch
doesn't cause regressions on systems where it currently works.  I doubt that
these are actually problems with the patch, but they need to be checked before I
can release esound with the patch applied.
Comment 6 Peter Åstrand 2005-10-02 19:30:00 UTC
IMHO, the first step to verify if this patch works on all systems is to apply it
to CVS. If the patch is left alone in this Bugzilla, no-one will try it. 

Or do you have some other plan for getting this patch merged? After all, it has
Priority: High (which I think is correct). 
Comment 7 Herman Bos 2006-03-11 10:34:38 UTC
Could this patch be merged into CVS please? 

Some people (me and many others) really need this. 

I hope something can be done with it besides letting it sit here. :-)
Comment 8 David Schleef 2006-08-08 17:45:59 UTC
Applied.