[47969] trunk/dports/graphics/jpeg/Portfile

[Joshua Root]

Ryan Schmidt wrote:
> 
> On Mar 11, 2009, at 16:29, mcalhoun at macports.org wrote:
> 
>> Revision: 47969
>>           http://trac.macports.org/changeset/47969
>> Author:   mcalhoun at macports.org
>> Date:     2009-03-11 14:29:23 -0700 (Wed, 11 Mar 2009)
>> Log Message:
>> -----------
>> jpeg: Prevent linker from finding wrong library and header files.
>> Fixes #16411 (maintainer timeout).
>> Revision not increased because the library should have built correctly
>> or not at all.
>>
>> Modified Paths:
>> --------------
>>     trunk/dports/graphics/jpeg/Portfile
>>
>> Modified: trunk/dports/graphics/jpeg/Portfile
>> ===================================================================
>> --- trunk/dports/graphics/jpeg/Portfile    2009-03-11 21:26:37 UTC
>> (rev 47968)
>> +++ trunk/dports/graphics/jpeg/Portfile    2009-03-11 21:29:23 UTC
>> (rev 47969)
>> @@ -40,6 +40,14 @@
>>  post-patch {
>>      system "cd ${worksrcpath} &&
>>          tar zxf ${distpath}/droppatch.tar.gz"
>> +
>> +    # Reorder link flags so that so that local -L options come first
>> (especially before -L${prefix}/lib)
>> +    #     (see http://trac.macports.org/ticket/16411).
>> +    reinplace
>> "s|\\(.*\\)\\(\$(LDFLAGS)\\)\\(.*\\)\\(\$(LDLIBS)\\)\\(.*\\)|\\1\\4\\3\\2\\5|"
>> ${worksrcpath}/makefile.cfg
> 
> This is hard to decipher. Could it be implemented as a patchfile
> instead, or would that be huge?

It could be made less indecipherable by using braces instead of quotes,
since no variables need to be expanded in the reinplace expression.

- Josh