[90899] trunk/dports/multimedia/ffmpeg-devel

[Ryan Schmidt]

On Mar 26, 2012, at 02:16, Andrea D'Amore wrote:
> On Sat, Mar 17, 2012 at 19:55,  <jeremyhu at macports.org> wrote:
>> Revision 90899 Author jeremyhu at macports.org Date 2012-03-17 11:55:38 -0700
> 
>> ffmpeg-devel: Update to current git master
> 
> There seems to be no related ticket, the maintainer should have been
> given the canonical 72 hours to commit.

The port is openmaintainer so updating without consulting the maintainer is fine.


> +# This is fixed in llvm-gcc trunk, but not in Apple's shipped llvm-gccs.
>> if {${configure.compiler} == "llvm-gcc-4.2"} {
>> -    configure.compiler gcc-4.2
>> -} elseif {${configure.compiler} == "clang"} {
>> -    depends_build-append port:apple-gcc42
>> -    configure.compiler apple-gcc-4.2
>> -    # base (as of 2.0.3) doesn't set cxx for apple-gcc-4.2
>> -    configure.cxx ${prefix}/bin/g++-apple-4.2
>> +    configure.compiler clang
>> }
> 
> This forces clang when llvm-gcc-4.2 is set but the latter is still a
> valid compiler on certain configurations of OSX/Xcode, isn't it?

llvm-gcc-4.2 is the default compiler on Xcode 4.0 and 4.1. ffmpeg apparently cannot be compiled with it. Therefore the port used to drop back to gcc-4.2. Jeremy changed this to prefer clang, which is generally the preferred compiler to use on Xcode 4+ these days.


> Is possibly #33735 [1] related to this clang forcing?

No; the ticket says the user is on Snow Leopard, and the log attached there shows the compiler being used is gcc-4.2.