Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 637914 - mozconfig-v6.57.eclass incorrectly list pulseaudio and apulse
Summary: mozconfig-v6.57.eclass incorrectly list pulseaudio and apulse
Status: RESOLVED INVALID
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Mozilla Gentoo Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-11-17 05:29 UTC by Eric Siegel
Modified: 2017-11-19 02:25 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.