From: | "Andrew Dunstan" <andrew(at)dunslane(dot)net> |
---|---|
To: | <pgman(at)candle(dot)pha(dot)pa(dot)us> |
Cc: | <tgl(at)sss(dot)pgh(dot)pa(dot)us>, <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] HEAD doesn't cope with libraries in non-default |
Date: | 2005-07-04 12:06:29 |
Message-ID: | 4950.24.211.165.134.1120478789.squirrel@www.dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Bruce Momjian said:
> Tom Lane wrote:
>> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
>> > Tom Lane wrote:
>> >> It appears that somebody has changed things so that the -L switches
>> >> appear after the -l switches (ie, too late). I'm too tired to
>> >> investigate now, but my money is on Autoconf 2.59 being the problem
>> >> ...
>>
>> > I wonder if it was this commit. I am attaching the patch so you can
>> > test to see if it fixes it. If it does, please let us know.
>>
>> > ! SHLIB_LINK := $(filter -L%, $(LDFLAGS)) $(SHLIB_LINK)
>> > ...
>> > ! SHLIB_LINK := $(SHLIB_LINK) $(filter -L%, $(LDFLAGS))
>>
>> Urgh. This was considered a good idea why exactly?
>
> I didn't like it myself but it fixed a problem with a build farm
> machine, and no one objected, so I applied it, but it seemed pretty
> strange to be reversing those. I will reverse the patch.
>
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.)
Alternatively, if we can't say --with-libraries=/foo/bar when /foo/bar
contains possibly conflicting libraries, that should be tested for at
configure time.
cheers
andrew
From | Date | Subject | |
---|---|---|---|
Next Message | Fabien COELHO | 2005-07-04 12:20:34 | Re: [PATCHES] Users/Groups -> Roles |
Previous Message | Hannu Krosing | 2005-07-04 11:26:54 | Re: PATCH to allow concurrent VACUUMs to not lock each |
From | Date | Subject | |
---|---|---|---|
Next Message | Fabien COELHO | 2005-07-04 12:20:34 | Re: [PATCHES] Users/Groups -> Roles |
Previous Message | Petr Jelinek | 2005-07-04 11:31:31 | Re: per user/database connections limit again |