Opened 12 years ago

Closed 10 years ago

#19619 closed defect (wontfix)

MPlayer: binary_codecs variant

Reported by: ryandesign (Ryan Schmidt) Owned by: ecronin (Eric Cronin)
Priority: Normal Milestone:
Component: ports Version: 1.7.1
Keywords: Cc: acho@…, simon@…
Port: MPlayer mplayer-devel

Description (last modified by ryandesign (Ryan Schmidt))

The binary_codecs variant of MPlayer did nothing at all, because it checked whether darwin_i386 or darwin_powerpc variants were set in order to decide what to do, but because no "platform darwin i386" or "platform darwin powerpc" section existed anywhere in the portfile, the variant did nothing. Those platform sections used to exist, but got removed somewhere along the way without consideration for the impact this would have on the binary_codecs variant.

I re-added empty platform statements in r50951 and now the variant fails with:

Error: Target org.macports.configure returned: configure failure: shell command " cd "/mp/var/macports/build/_Users_rschmidt_macports_dports_multimedia_MPlayer/work/MPlayer-1.0rc2" && ./configure --prefix=/mp --with-extraincdir=/mp/include/lzo:/mp/include --with-extralibdir=/mp/lib:/mp/lib/samba3 --datadir=/mp/share/MPlayer --confdir=/mp/etc/MPlayer --mandir=/mp/share/man --enable-gif --enable-png --enable-jpeg --enable-liblzo --enable-libvorbis --enable-mad --disable-faad-external --enable-largefiles --disable-smb --disable-live --disable-dvdnav --disable-dvdread --disable-cdparanoia --disable-freetype --disable-fontconfig --disable-unrarlib --disable-fribidi --disable-enca --disable-libcdio --disable-xvid --disable-x264 --disable-speex --disable-theora --disable-faac --disable-ladspa --disable-libdv --disable-toolame --disable-twolame --disable-xmms --disable-musepack --disable-sdl --disable-aa --disable-caca --disable-x11 --disable-gl --disable-arts --disable-esd --disable-lirc --with-codecsdir=/mp/share/MPlayer/codecs --enable-macosx --enable-macosx-finder-support --enable-macosx-bundle --enable-qtx " returned error 1
Command output: Unknown parameter: --with-codecsdir=/mp/share/MPlayer/codecs

That's on Mac OS X 10.5.6 Intel with Xcode 3.1.2 and MacPorts 1.7.1.

I found this as a result of a post to the mailing list, and in my response I discussed the issue further.

Change History (6)

comment:1 Changed 12 years ago by ryandesign (Ryan Schmidt)

Description: modified (diff)

comment:2 Changed 12 years ago by ecronin (Eric Cronin)

Port: mplayer-devel added

Oops, deleting the platforms was my bad, 1.0rc1 required disabling inline assembly on x86 and when that was fixed in 1.0rc2 I forgot that binary_codecs also needed them. Their current position right above the variant and unusualness of doing nothing are self-documenting, but an explicit reminder that they're required is probably also good incase a similar situation happens in the future...

The plugins flag changed to --codecsdir, but I believe as of rc2 the internal wine code needed to load DLLs on x86 was not working. mplayer-devel is probably a better port to be working with to re-enable plugins now that it exists (assuming the wine loader is fixed in subversion), and on this old MPlayer port the x86 target just removed.

comment:3 Changed 12 years ago by acho@…

Cc: acho@… added

Cc Me!

comment:4 Changed 12 years ago by acho@…

Added to mplayer-devel in r50967.

comment:5 Changed 12 years ago by simon@…

Cc: simon@… added

Cc Me!

comment:6 Changed 10 years ago by jmroot (Joshua Root)

Resolution: wontfix
Status: newclosed

This variant was removed in r59989 and r59990.

Note: See TracTickets for help on using tickets.