Summary: | media-sound/pulseaudio - padsp hardcodes LD_PRELOAD to lib64 - will not work for 32 bit programs | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Marc Joliet <marcec> |
Component: | Current packages | Assignee: | Gentoo Linux Gnome Desktop Team <gnome> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | marcec |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | AMD64 | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
Marc Joliet
2013-09-28 13:57:12 UTC
Still valid with 4.99.4? If still valid looks like an upstream problem to me that should be reported to bugzilla.freedesktop.org and, once it's done, you could paste the link here for letting us to track the issue (In reply to Pacho Ramos from comment #1) > Still valid with 4.99.4? If still valid looks like an upstream problem to me > that should be reported to bugzilla.freedesktop.org and, once it's done, you > could paste the link here for letting us to track the issue Sorry for the belated reply, I just upgraded to PA 5.0 and the library path is still hard-coded in padsp. I'll report upstream when I get the chance, however it is pretty much a non-issue for me now. Welp, I never got around to reporting this upstream, but I noticed that they did fix it in 11.0. See the heading "Support for 32-bit applications on 64-bit systems in padsp" at https://www.freedesktop.org/wiki/Software/PulseAudio/Notes/11.0/. Would it make sense for the fix to be enabled in Gentoo? Not sure what your suggest, we have 11.1 in tree is there anything else we need to do ? Yes, https://www.freedesktop.org/wiki/Software/PulseAudio/Notes/11.0/#notesforpackagers talks what we need to do (pass an extra configure argument on glibc systems only), but maybe you already did that. re-opening for actually getting this fixed at some point. Currently will stabilize 11.1 without pushing and including this fix (I don't feel comfortable to immediately stabilize it). Hopefully we get to including a fix in a revbump though based on this reopened report soon. After applying ebuild changes for glibc, I'm now getting this from LD_DEBUG=libs when using padsp with a 32bit binary: calling init: /usr/lib32/pulseaudio/libpulsedsp.so /usr/lib32/pulseaudio/libpulsedsp.so: error: symbol lookup error: undefined symbol: __padsp_disabled__ (fatal) and lib64 when using a 64bit binary. So I guess it works now, that __padsp_disabled__ symbol lookup error looks normal (PA daemon actually has code to trigger things based on that symbol being present or not, and if present, containing a bitmask about padsp or something). I have not tested with any actual program that uses OSS and actually needs padsp to function, not 64bit nor 32bit. Please test and confirm after the 12.2 version is available in your tree with the configure changes. There will be an automatic comment added by a bot once the bump is available in git, will need a bit more time to finalize other fixes and push. The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=2f79bc944a2dedb1e8ca2b56c53ca9c2a5354aa8 commit 2f79bc944a2dedb1e8ca2b56c53ca9c2a5354aa8 Author: Mart Raudsepp <leio@gentoo.org> AuthorDate: 2018-07-27 10:11:04 +0000 Commit: Mart Raudsepp <leio@gentoo.org> CommitDate: 2018-07-27 11:44:28 +0000 media-sound/pulseaudio: fix padsp OSS wrapper for non-native multilib Pass the new (in v11) configure argument with glibc per upstream release notes. Closes: https://bugs.gentoo.org/486328 Package-Manager: Portage-2.3.43, Repoman-2.3.10 media-sound/pulseaudio/pulseaudio-12.2.ebuild | 5 +++++ 1 file changed, 5 insertions(+) |