[100993] trunk/dports/devel/csu/Portfile

[Michael Feiri]

On 03.01.2013, at 09:40, Ryan Schmidt <ryandesign at macports.org> wrote:

> 
> On Jan 2, 2013, at 19:57, mfeiri at macports.org wrote:
> 
>> Revision: 100993
>>         https://trac.macports.org/changeset/100993
>> Author:   mfeiri at macports.org
>> Date:     2013-01-02 17:57:20 -0800 (Wed, 02 Jan 2013)
>> Log Message:
>> -----------
>> csu: more specific architecture support, including 64-bit on darwin 9
>> 
>> Modified Paths:
>> --------------
>>   trunk/dports/devel/csu/Portfile
>> 
>> Modified: trunk/dports/devel/csu/Portfile
>> ===================================================================
>> --- trunk/dports/devel/csu/Portfile	2013-01-03 01:46:13 UTC (rev 100992)
>> +++ trunk/dports/devel/csu/Portfile	2013-01-03 01:57:20 UTC (rev 100993)
>> @@ -21,8 +21,13 @@
>> default_variants        +universal
>> build.env-append        CC='${configure.cc} -Wl,-new_linker'
>> 
>> +
>> variant universal {
>> -    build.env-append    RC_ARCHS='${universal_archs}'
>> +    if {${os.major} == 9} {
>> +        build.env-append    RC_ARCHS='ppc ppc64 i386 x86_64'
>> +    } else {
>> +        build.env-append    RC_ARCHS='i386 x86_64'
>> +    }
>> }
> 
> This doesn't seem right. Instead, you should be setting universal_archs to the list of architectures you're going to build for (if it is not possible to accept the list of architectures the user requested in their macports.conf). Otherwise the list of architectures shown in "port -v installed csu" will not agree with the list of architectures for which the port has actually been compiled.

Okay, changed in r104698.

Regards, Michael