[43775] trunk/dports/audio/mpg123/Portfile

[Joshua Root]

Ryan Schmidt wrote:
> 
> On Dec 14, 2008, at 15:17, jmr at macports.org wrote:
> 
>> Revision: 43775
>>           http://trac.macports.org/changeset/43775
>> Author:   jmr at macports.org
>> Date:     2008-12-14 13:17:09 -0800 (Sun, 14 Dec 2008)
>> Log Message:
>> -----------
>> mpg123: platform 'powerpc' -> 'ppc'
>>
>> Modified Paths:
>> --------------
>>     trunk/dports/audio/mpg123/Portfile
>>
>> Modified: trunk/dports/audio/mpg123/Portfile
>> ===================================================================
>> --- trunk/dports/audio/mpg123/Portfile    2008-12-14 18:52:09 UTC (rev
>> 43774)
>> +++ trunk/dports/audio/mpg123/Portfile    2008-12-14 21:17:09 UTC (rev
>> 43775)
>> @@ -27,6 +27,6 @@
>>      configure.args-append --with-cpu=x86
>>  }
>>
>> -platform macosx powerpc {
>> +platform macosx ppc {
>>      configure.args-append --with-cpu=altivec
>>  }
> 
> 
> We have 60 occurrences of something like "platform powerpc" in the ports
> tree and only 8 occurrences of something like "platform ppc"...... Do
> both work? The Guide only shows "powerpc".
> 
> http://guide.macports.org/#reference.variants.platform

You're right, powerpc is correct, ppc is not. The change was motivated
by the fact that the macosx_powerpc variant was not being selected on a
ppc machine. However, it turns out that the actual problem is that
'platform $plat $arch' doesn't get selected automatically when $plat is
not the same as ${os.platform}. This currently applies to macosx and
puredarwin.

The issue was clouded a little by the fact that 'platform $plat' does
get selected in the same situation because there are some special cases
in portmain.tcl.

- Josh