[28197] trunk/dports/palm/hrsh2csv/Portfile

[Ryan Schmidt]

On Aug 27, 2007, at 18:09, N_Ox wrote:

> Le 28 août 07 à 00:42, Simon Ruderich a écrit :
>
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA512
>>
>> source_changes at macosforge.org wrote:
>>> Revision: 28197
>>>           http://trac.macosforge.org/projects/macports/changeset/ 
>>> 28197
>>> Author:   css at macports.org
>>> Date:     2007-08-24 06:18:36 -0700 (Fri, 24 Aug 2007)
>>>
>>> Log Message:
>>> -----------
>>> Create a powerpc platform variant that removes the endian  
>>> conversion. This
>>> fixes hrsh2csv to function properly on Intel Macs. (#12555)
>>>
>>> Modified Paths:
>>> --------------
>>>     trunk/dports/palm/hrsh2csv/Portfile
>>>
>>> Modified: trunk/dports/palm/hrsh2csv/Portfile
>>> ===================================================================
>>> --- trunk/dports/palm/hrsh2csv/Portfile	2007-08-24 13:17:34 UTC  
>>> (rev 28196)
>>> +++ trunk/dports/palm/hrsh2csv/Portfile	2007-08-24 13:18:36 UTC  
>>> (rev 28197)
>>> @@ -3,6 +3,7 @@
>>>  PortSystem		1.0
>>>  name			hrsh2csv
>>>  version			1.3
>>> +revision		1
>>>  description		converts Hours pdb files into CSV text files
>>>  long_description	Hours is a time-tracking utility for the PalmOS. \
>>>  	This utility converts Hours for PalmOS database files into       \
>>> @@ -16,8 +17,13 @@
>>>
>>>  use_configure		no
>>>  post-extract {
>>> -	reinplace "s|-O2 -D__MAC2PC_CONVERSION__|-O2|" ${worksrcpath}/ 
>>> Makefile
>>>  	reinplace "s|prefix = /usr/local|prefix = ${destroot}$ 
>>> {prefix}|" ${worksrcpath}/Makefile
>>>  }
>>> +
>>> +# For PowerPC, remove the endian conversion since PDBs are big  
>>> endian.
>>> +platform powerpc {
>>> +	reinplace "s|-O2 -D__MAC2PC_CONVERSION__|-O2|" ${worksrcpath}/ 
>>> Makefile
>>> +}
>>> +
>>>  destroot.args		mandir=${destroot}${prefix}/share/man/man1
>>
>> I don't know why, but it looks like macports is trying to execute the
>> "reinplace" even if only something like "port info" is called on this
>> file. So it should be moved to post-configure, but I don't know  
>> how this
>> is done.
>>
>> Simon
>
> The reinplace call should be in a post-patch block.

We discussed this a couple days ago on the list and there was info in  
the bug about it:

http://trac.macosforge.org/projects/macports/ticket/12555