Re: [PATCHES] HEAD doesn't cope with libraries in non-default

This patch seems to have broken builds on Windows and other boxes (e.g. 
buildfarm's octopus, a FreeBSD box). Maybe this should be reverted until 
we find a more robust solution :-(

cheers

andrew

Bruce Momjian wrote:

>Patch applied.  Thanks.
>
>---------------------------------------------------------------------------
>
>
>Peter Eisentraut wrote:
>  
>
>>I wrote:
>>    
>>
>>>So either we code up some intelligence to put the "C" in the right
>>>position or we have to pass down "A B" and "D" separately from the
>>>main makefile.
>>>      
>>>
>>The following patch might just do the former.  Please try it out.
>>
>>
>>diff -ur ../cvs-pgsql/src/Makefile.shlib ./src/Makefile.shlib
>>--- ../cvs-pgsql/src/Makefile.shlib     2005-07-04 16:32:57.000000000 +0200
>>+++ ./src/Makefile.shlib        2005-07-05 22:02:10.556162778 +0200
>>@@ -240,7 +240,7 @@
>>   SHLIB_LINK           += -ltermcap -lstdc++.r4 -lbind -lsocket -L/boot/develop/lib/x86
>> endif
>>
>>-SHLIB_LINK := $(filter -L%, $(LDFLAGS)) $(SHLIB_LINK)
>>+SHLIB_LINK := $(filter -L%, $(SHLIB_LINK)) $(filter -L%, $(LDFLAGS)) $(filter-out -L%, $(SHLIB_LINK))
>> ifeq ($(enable_rpath), yes)
>> SHLIB_LINK += $(rpath)
>> endif
>>    
>>