[56907] trunk/dports/emulators/nonpareil

[Ryan Schmidt]

On Sep 3, 2009, at 09:05, Dipl. Inform. Martin Krischik wrote:

> Am 03.09.2009 um 15:40 schrieb Ryan Schmidt:
>
>>> Added: trunk/dports/emulators/nonpareil/files/osx.patch
>>> ===================================================================
>>> --- trunk/dports/emulators/nonpareil/files/ 
>>> osx.patch	                        (rev 0)
>>> +++ trunk/dports/emulators/nonpareil/files/osx.patch	2009-09-03  
>>> 13:10:50 UTC (rev 56907)
>>
>>> +!     pkg_config_cmd = '/opt/local/bin/pkg-config'
>>> +  pkg_config_cmd += ' --cflags --libs '
>>> +
>>> +! sdl_pkg_config_cmd = '/opt/local/bin/sdl-config --cflags --libs'
>>
>> You mustn't hardcode /opt/local; you must make it use ${prefix}.
>
> Damm another one escaped me. Of course that is a patch file - won't  
> I need to use reinplace in the pre-patch phase instead?

You could use "@PREFIX@/bin" in the patch, and then reinplace @PREFIX@  
with ${prefix} in post-patch.

It would probably work fine without the patch, though, since ${prefix}/ 
bin is already in the binpath by default. Only reason to use such a  
patch would be if you expect users to modify the binpath -- something  
we don't recommend they do.