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

From Peter Eisentraut
Subject Re: [PATCHES] HEAD doesn't cope with libraries in non-default
Date
Msg-id 200507041758.28526.peter_e@gmx.net
Whole thread Raw
Responses Re: [PATCHES] HEAD doesn't cope with libraries in non-default
Re: [PATCHES] HEAD doesn't cope with libraries in non-default
List pgsql-hackers
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
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/


pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: [PATCHES] Disable page writes when fsync off, add GUC
Next
From: Alvaro Herrera
Date:
Subject: New warnings in plgsql/src/scan.l