Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 200858 - media-libs/libmp4v2-1.5.0.1 can't be configured with dev-lang/nasm-2.00
Summary: media-libs/libmp4v2-1.5.0.1 can't be configured with dev-lang/nasm-2.00
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Library (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Olivier Crete (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-11-30 19:31 UTC by Víctor Enríquez
Modified: 2008-01-22 16:52 UTC (History)
9 users (show)

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


Attachments
Changes -r to -v in configure script (libmp4v2-nasm-2.00.patch,469 bytes, patch)
2007-12-05 03:47 UTC, darklegion
Details | Diff
Updated ebuild (libmp4v2-1.5.0.1.ebuild,1.78 KB, text/plain)
2007-12-05 03:48 UTC, darklegion
Details
libmp4v2-1.5.0.1.ebuild.diff (libmp4v2-1.5.0.1.ebuild.diff,764 bytes, text/plain)
2007-12-06 02:36 UTC, Lars Wendler (Polynomial-C) (RETIRED)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Víctor Enríquez 2007-11-30 19:31:15 UTC
When you try to install libmp4v2 with nasm-2.0 it will fail, repeating this message forever (infinite loop):

checking for nasm... true
checking nasm version...... nasm: error: unrecognised option `-r'
nasm: error: no input file specified
type `nasm -h' for help
util/testnasm.sh: line 8: test: =: unary operator expected
util/testnasm.sh: line 8: test: =: unary operator expected
util/testnasm.sh: line 8: test: =: unary operator expected
util/testnasm.sh: line 8: test: =: unary operator expected
util/testnasm.sh: line 8: test: =: unary operator expected
util/testnasm.sh: line 8: test: =: unary operator expected
util/testnasm.sh: line 8: test: =: unary operator expected
util/testnasm.sh: line 8: test: =: unary operator expected
util/testnasm.sh: line 8: test: =: unary operator expected


Reproducible: Always
Comment 1 Rafał Mużyło 2007-12-01 12:24:15 UTC
Well, you should treat this as an upstream bug.
nasm 1.98 listed -r option as obsolete (-v preferred).
Comment 2 James 2007-12-01 13:32:33 UTC
Help! this has made it into amd64 stable.
Comment 3 darklegion 2007-12-05 03:47:16 UTC
Created attachment 137770 [details, diff]
Changes -r to -v in configure script
Comment 4 darklegion 2007-12-05 03:48:53 UTC
Created attachment 137771 [details]
Updated ebuild
Comment 5 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2007-12-06 02:36:47 UTC
Created attachment 137857 [details]
libmp4v2-1.5.0.1.ebuild.diff

Better approach which fixes configure.in and not configure, uses eautoreconf from autotools.eclass, doesn't need a patch and will only get applied when >=nasm-2 is installed.
Though I don't know if elibtoolize is still needed when eautoreconf was invoked before so I left that untouched in the ebuild.
Comment 6 Andrés Becerra Sandoval 2007-12-11 16:09:53 UTC
(In reply to comment #5)
> Created an attachment (id=137857) [edit]
> libmp4v2-1.5.0.1.ebuild.diff
> 

I have tested your fix in ~x86 successfully!

 Andrés
Comment 7 Billy DeVincentis 2007-12-12 03:53:44 UTC
Lars,  what  exactly  is  the  difference  between  your  patch  and  the  other  patch  that  has  an  updated  ebuild?  If  you  are  using  nasm 2  from  testing  does  it  matter  the  approach,  His  ebuild  emerged  fine.
Comment 8 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2007-12-14 12:45:30 UTC
Like I said in my previous post. My patch fixes the real source of the problem (Makefile.in) and not the autotools generated Makefile. Furthermore Makefile.in only needs the fix when >=nasm-2 is installed as otherwise libmp4v2 would break when there is still <nasm-2 installed. Finally the fix is (imho) rather trivially done with sed from inside the ebuild. No need to add an extra patch into the portage-tree.
Comment 9 Ferry 2008-01-06 15:08:20 UTC
Any chance of committing? Issue still exists and patch runs fine here (used the one from Lars).
Comment 10 Steve Dibb (RETIRED) gentoo-dev 2008-01-11 17:54:57 UTC
Fixed in -r3
Comment 11 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2008-01-15 15:53:52 UTC
Marked as fixed but there were no update on media-libs/libmp4v2 for the last seven weeks...
Can anyone please reopen this bug until it's really fixed?
Comment 12 Kyle Elbert 2008-01-18 12:56:34 UTC
Experienced bug. Patch and updated ebuild work, but they're not in portage ....
Comment 13 David Sveningsson 2008-01-22 12:08:09 UTC
Could someone update the ebuild in portage?
Comment 14 Jakub Moc (RETIRED) gentoo-dev 2008-01-22 12:24:38 UTC
(In reply to comment #10)
> Fixed in -r3

??? 

Comment 15 Dennis Schridde 2008-01-22 13:24:24 UTC
(In reply to comment #10)
> Fixed in -r3
Funny thing is there is no -rX at all in portage... ;)

[I] media-libs/libmp4v2
     Available versions:  1.5.0.1
Comment 16 Olivier Crete (RETIRED) gentoo-dev 2008-01-22 16:26:54 UTC
-r1 is in the tree.
Comment 17 Dennis Schridde 2008-01-22 16:37:25 UTC
Did the tree move away from sources.gentoo.org/gentoo-x86 ?

At least http://sources.gentoo.org/viewcvs.py/gentoo-x86/media-libs/libmp4v2/ doesn't show any -r1 at the time of this writing...
Comment 18 Olivier Crete (RETIRED) gentoo-dev 2008-01-22 16:52:33 UTC
be patient, its there now..