Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 361879 - media-sound/lame USE=-mmx - configure handles --disable-nasm improperly
Summary: media-sound/lame USE=-mmx - configure handles --disable-nasm improperly
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal minor (vote)
Assignee: Gentoo Sound Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-04 08:32 UTC by citiesom
Modified: 2012-01-20 20:33 UTC (History)
1 user (show)

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


Attachments
workaround patch for ebuild (lame-3.98.4-r1.ebuild,1.25 KB, patch)
2011-05-08 05:47 UTC, Dillon
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description citiesom 2011-04-04 08:32:58 UTC
Line 28255
 if test "${enable_nasm+set}" = set; then
determines whether the assembly gets compiled in. However, using ./configure --disable-nasm will also compile in the assembly since enable_nasm=no still lets ${enable_nasm+set} equal 'set'. This in turn breaks the ebuild; the assembly is always compiled in using portage.

Reproducible: Always

Steps to Reproduce:
1. euse -D mmx #disables mmx/nasm
2. emerge media-sound/lame
Actual Results:  
mmx/nasm are enabled.
Comment 1 Dillon 2011-05-07 09:15:29 UTC
This creates a TEXTREL on hardened too. Looking at the IUSE of various incidentally affected packages shows us that phonon,vlc, and mplayer are broken.
Comment 2 Dillon 2011-05-07 09:16:15 UTC
(In reply to comment #1)
> This creates a TEXTREL on hardened too. Looking at the IUSE of various
> incidentally affected packages shows us that phonon,vlc, and mplayer are
> broken.
sorry,
If USE=mp3 is in /etc/make.conf
Comment 3 Dillon 2011-05-08 05:47:38 UTC
Created attachment 272451 [details, diff]
workaround patch for ebuild
Comment 4 Samuli Suominen (RETIRED) gentoo-dev 2012-01-20 20:33:55 UTC
I've added a workaround in the 3.99.3 ebuild for this.    thanks guys