Changes between Initial Version and Version 2 of Ticket #33489


Ignore:
Timestamp:
Mar 6, 2012, 1:04:11 AM (12 years ago)
Author:
ecronin (Eric Cronin)
Comment:

First, you probably should be looking at mplayer-devel or mplayer2. The MPlayer port is dead and should just be removed from ports since all it seems to do is cause confusion (it still builds most of the time which is why I'm hesitant to just replaced_by it in case someone really does think a 5 year old completely unsupported by upstream release candidate is better than a slightly less unsupported by upstream port with -devel in its name)

Regarding that comment, the "let autodetect do its magic" part means that explicit --enable-foo or --enable-foo=/opt/local do not do the right thing with (ancient) mplayer's ./configure. To enable a feature you really do need to remove the --disable-foo flag but just let ./configure autodetect that foo is present, instead of adding --enable-foo like you do in normal autoconf'd programs. But the removing of --disable-foo only happens inside variant blocks where the necessary depends are added, e.g.:

variant aa \
    description {Enable animated ASCII art video output} {
    depends_lib-append      port:aalib
    configure.args-delete   --disable-aa
}

This is obviously very brittle, and it looks like some new knobs got added at some point for libraries that didn't exist in macports years ago. ffmpeg is odd since it used to always use a private internal copy and not link against the system version... In fact in comment 2 I don't see ffmpeg listed there. Are you sure it was complaining about ffmpeg and not just opencore-amr?

I'm cc'ing the mplayer-devel and mplayer2 maintainers since they may have similar issues where new autodetected libraries have been added to ./configure that need to be explicitly disabled in the Portfile

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #33489

    • Property Cc acho@… cal@… added
  • Ticket #33489 – Description

    initial v2  
    33That is not exactly a surprise, given that the Portfile contains the following misguided comment:
    44
     5{{{
    56# MPlayer autodetects many support libs.  To prevent undeclared
    67# dependencies, explicitly disable everything optional first.
    78# Later, let autodetect do its magic not explicit --enable
     9}}}
    810
    911Letting autodetect "do its magic" without explicitly --enabling