[Bug 195221] Review Request: pulseaudio: Improved Linux sound server

bugzilla at redhat.com bugzilla at redhat.com
Fri Jul 21 11:09:52 UTC 2006


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: pulseaudio: Improved Linux sound server


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=195221





------- Additional Comments From drzeus-bugzilla at drzeus.cx  2006-07-21 07:01 EST -------
(In reply to comment #16)
> I had no problems building in rawhide, but rpmlint is very unhappy:
> 
> Tons of rpath warnings:
>   E: pulseaudio binary-or-shlib-defines-rpath  
> /usr/lib64/pulse-0.9/modules/libprotocol-esound.so
> ['/usr/lib64/pulse-0.9/modules/', '/usr/lib64']
> and 69 others.  The usual libtool hack doesn't work.

This is a libtool/gcc misfeature. I talked to Ralf Wildenhues about it, but he
didn't have any way of solving it (short of bastardising libtool). It's just the
way things are on multi-arch for now.

> 
> This looks like a script but isn't executable:
>   E: pulseaudio non-executable-script /etc/pulse/default.pa 0644
> 

The hash bang is mostly there to show that it can be used that way. As it is
located in /etc, I think we should keep the execute bit off.

> Some setuid bits:
>   E: pulseaudio setuid-binary /usr/bin/pulseaudio root 04755
>   E: pulseaudio non-standard-executable-perm /usr/bin/pulseaudio 04755
> which will need to be discussed to make sure we're not getting into any security
> issues.

Sure. Pulse only uses its root privileges to change to realtime scheduling, then
drops them. Access control is based on who is in the 'realtime' group.

> 
> Some no-documentation warnings:
>   W: pulseaudio-devel no-documentation
>   W: pulseaudio-lib-glib no-documentation
>   W: pulseaudio-lib-zeroconf no-documentation
>   W: pulseaudio-module-alsa no-documentation
>   W: pulseaudio-module-x11 no-documentation
>   W: pulseaudio-module-zeroconf no-documentation
>   W: pulseaudio-utils no-documentation
> which are probably OK but I'll have to check.
> 

The docs are in the other packages.

> This is problematic:
>   E: pulseaudio-utils invalid-soname /usr/lib64/libpulsedsp.so libpulsedsp.so
> but unfortunately I'm not entirely sure how to fix it.

It's not a "real" lib, so it has no use for versioning. It's the active part of
an LD_PRELOAD hack to provide OSS emulation. As such, the error is incorrect and
the lib is just fine.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.




More information about the Fedora-package-review mailing list