Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 439184 - app-emulation/wine (several versions): pulseaudio patchset ignores useflag
Summary: app-emulation/wine (several versions): pulseaudio patchset ignores useflag
Status: RESOLVED INVALID
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Wine Maintainers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-21 16:46 UTC by Ramon Klass
Modified: 2012-10-22 19:58 UTC (History)
1 user (show)

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


Attachments
patch to retroactively only patch when the useflag was set (wine-optional_pulse.patch,6.93 KB, patch)
2012-10-21 16:48 UTC, Ramon Klass
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ramon Klass 2012-10-21 16:46:24 UTC
one-liner, but in many ebuilds: the pulseaudio patchset gets applied no matter if use pulseaudio is set or not. Do I misunderstand the purpose of useflags?


Reproducible: Always

Steps to Reproduce:
emerge wine with USE="-pulseaudio"
Actual Results:  
pulseaudio patches get applied either way

Expected Results:  
would have expected the pulseaudio patchset to be left out depending on use
Comment 1 Ramon Klass 2012-10-21 16:48:20 UTC
Created attachment 327088 [details, diff]
patch to retroactively only patch when the useflag was set
Comment 2 Jeroen Roovers (RETIRED) gentoo-dev 2012-10-22 04:59:14 UTC
Comment on attachment 327088 [details, diff]
patch to retroactively only patch when the useflag was set

All patches should ideally apply generally (not conditionally) and add changes to the source that work out configuring specifically for the feature or enhancement that is being added.
Comment 3 SpanKY gentoo-dev 2012-10-22 19:58:47 UTC
(In reply to comment #2)

more specifically, applying the patch does not itself enable pulseaudio support.  only the configure flag (which is controlled by the USE flag) later on enables support.