[72457] trunk/dports/devel/ld64/Portfile

[Rainer Müller]

On 2010-10-16 14:50 , Ryan Schmidt wrote:
> On Oct 16, 2010, at 04:53, mfeiri at macports.org wrote:
> 
>> Revision: 72457
>>          http://trac.macports.org/changeset/72457
>> Author:   mfeiri at macports.org
>> Date:     2010-10-16 02:52:57 -0700 (Sat, 16 Oct 2010)
>> Log Message:
>> -----------
>> move llvm-config to pre-configure
> 
> 
>> +    pre-configure {
>> +        build.env       OTHER_CFLAGS='[exec ${prefix}/bin/llvm-config --cflags]'
>> +        destroot.env    OTHER_CFLAGS='[exec ${prefix}/bin/llvm-config --cflags]'
>> +    }
> 
>> +    pre-configure {
>> +        build.env       OTHER_CFLAGS='[exec ${prefix}/bin/llvm-config --cflags]'
>> +        destroot.env    OTHER_CFLAGS='[exec ${prefix}/bin/llvm-config --cflags]'
>> +    }
> 
> Actually, that's not going to be optimal. build.env should be set in pre-build, and destroot.env should be set in pre-destroot. Otherwise you cause problems for someone who explicitly runs "sudo port configure ld64" and then wants to continue building and destrooting it later.

Gah, you are absolutely right.
Sorry, Michael, my earlier advice was wrong.

Rainer