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

From Jim C. Nasby
Subject Re: [PATCHES] HEAD doesn't cope with libraries in non-default
Date
Msg-id 20050705174355.GB49841@decibel.org
Whole thread Raw
In response to Re: [PATCHES] HEAD doesn't cope with libraries in non-default  (Peter Eisentraut <peter_e@gmx.net>)
Responses Re: [PATCHES] HEAD doesn't cope with libraries in non-default  (Peter Eisentraut <peter_e@gmx.net>)
List pgsql-hackers
On Mon, Jul 04, 2005 at 05:58:27PM +0200, Peter Eisentraut wrote:
> Andrew Dunstan wrote:
> > I was also slightly dubious about it. However, we do still need to
> > solve the problem that the patch addressed. Buildfarm members
> > platypus and cuckoo are currently failing because dblink is picking
> > up the wrong libpq (and it appears that incorrect libraries are also
> > being picked up in the ecpg libraries, although this isn't causing a
> > buildfarm failure.)
> 
> We have four pieces of information when linking a shared library:
> 
> B: in-tree libraries that we might need (in case of ecpglib: libpq)
> A: path to those in-tree libraries

Is A even represented in the build at all right now? ISTM it's not, so
simply adding it in front of C might suffice. What would be a reasonable
way to add that to the makefiles?

> D: external libraries that we might need (in case of ecpglib in my case: 
> -lcrypt -lm)
> C: path to those external libraries (e.g., /usr/local/lib)
> 
> On the linker command line, we need this information in one of the 
> following two orders:
> 
> A B C D
> A C B D
> 
> The Makefile.shlib receives from the respective main makefile "A B D" in 
> SHLIB_LINK and would have to insert "C" in the middle somewhere.  
> Currently, the actual behavior is "C A B D" and the failed patch wanted 
> to do "A B D C", both of which are wrong.
> 
> 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.
> 
> -- 
> Peter Eisentraut
> http://developer.postgresql.org/~petere/
> 
> ---------------------------(end of broadcast)---------------------------
> TIP 7: don't forget to increase your free space map settings
> 

-- 
Jim C. Nasby, Database Consultant               decibel@decibel.org 
Give your computer some brain candy! www.distributed.net Team #1828

Windows: "Where do you want to go today?"
Linux: "Where do you want to go tomorrow?"
FreeBSD: "Are you guys coming, or what?"


pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: timezone changes break windows and cygwin
Next
From: Tom Lane
Date:
Subject: Re: pgcrypto 3des failure, OpenSSL 0.9.8, Solaris 9/sparc