Re: make check error on -HEAD

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, Devrim GUNDUZ <devrim(at)gunduz(dot)org>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: make check error on -HEAD
Date: 2004-10-31 21:52:00
Message-ID: 41855E80.2010707@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Tom Lane wrote:

>Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
>
>
>>Tom Lane wrote:
>>
>>
>>>Oh, so you are using yesterday's libpq.so shared library ;-)
>>>
>>>I am not sure there is any way around that except to go ahead and
>>>install today's libpq. pg_regress can't do much more than set
>>>LD_LIBRARY_PATH, and evidently that's not enough to make the Linux
>>>dynamic loader take the version of libpq.so that's in the temp
>>>installation rather than the one you previously installed.
>>>
>>>
>
>
>
>>Yep, I saw the same thing here and make install fixed it.
>>
>>
>
>I looked at this a bit more and found that on Linux, the dynamic
>loader is documented to search "rpath" before LD_LIBRARY_PATH;
>so had we not specified an rpath when building the psql executable,
>pg_regress would have worked as intended. Sounds like BSD is the same.
>
>Now, not specifying rpath seems like a sure loss for every context
>except "make check" with an uninstalled version. So I'm afraid we have
>to live with it. It might be worthwhile for build-farm builds to use
>"configure --disable-rpath", if they want to "make check" without
>installing first.
>
>
>
>

The build-farm script removes the installation directory (we don't use
the default, of course) after each run, so the library won't ever be
found in the rpath during "make check", regardless of this setting.

cheers

andrew

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Joe Conway 2004-10-31 21:54:29 Re: Version defines
Previous Message Joe Conway 2004-10-31 21:51:24 Re: array_to_column function