Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 637914

Summary: mozconfig-v6.57.eclass incorrectly list pulseaudio and apulse
Product: Gentoo Linux Reporter: Eric Siegel <eric>
Component: EclassesAssignee: Mozilla Gentoo Team <mozilla>
Status: RESOLVED INVALID    
Severity: normal CC: tsmksubc
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---

Description Eric Siegel 2017-11-17 05:29:26 UTC
I wanted to build firefox-57 with USE=pulseaudio, but (for whatever reason) I chose to not install pulseaudio and use apulse instead.

I added media-sound/pulseaudio to package.mask and tried to emerge firefox-57 with USE=pulseaudio and got an error that firefox needed media-sound/pulseaudio.

I tried to figure out why, since it should accept either media-sound/pulseaudio or media-sound/apulse.

I found a few issues/weirdnesses in mozconfig-v6.57.eclass.

At line 106 (inside RDEPEND=""), there is:

pulseaudio? ( || ( media-sound/pulseaudio
    >=media-sound/apulse-0.1.9 ) )

This is correct, but the down at line 169 (inside DEPEND=""), there is:

pulseaudio? ( media-sound/pulseaudio )

Why is this there?  The even weirder thing is that right below that on line 175 (inside RDEPEND+="), you see:

    pulseaudio? ( || ( media-sound/pulseaudio
        >=media-sound/apulse-0.1.9 ) )

I'm pretty sure this block should only be added *once* and I think it should let you use media-sound/pulseaudio or media-sound/apulse-0.1.9.
Comment 1 Jory A. Pratt gentoo-dev 2017-11-19 02:25:42 UTC
The rdep is not the same as the build dep. The build dep is required for the headers, The dep is correct and will not be changed.