From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com> |
Cc: | pgsql-committers(at)postgresql(dot)org, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: pgsql: Record full paths of programs sought by "configure". |
Date: | 2017-08-07 20:20:46 |
Message-ID: | 12321.1502137246@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com> writes:
> On 08/07/2017 04:07 PM, Tom Lane wrote:
>> Sorry, I was imprecise. What I'm suggesting is that you drop the
>> runtime PATH-foolery and instead put this in configure's environment:
>>
>> PROVE=$perlpathdir/prove
>>
>> Otherwise you're basically lying to configure about what you're going
>> to use, and that's always going to break eventually.
> Hmm, you're saying this should work now? OK, I'll try it when I get a
> minute to spare.
I'm pretty sure it's always worked, at least in the sense that you could
override what configure would put into Makefile.global that way. I'm not
quite clear on whether providing an exact path to "prove" there is enough
to fix your problem. If we have any places where we need to invoke the
corresponding version of "perl", then we have more things to fix.
>> Hm, yeah, the IPC::Run test would need to deal with this as well.
>> A PROVE_PERL environment variable is one way. Or maybe simpler,
>> just skip the probe for IPC::Run if PROVE has been specified
>> externally; assume the user knows what he's doing in that case.
> WFM
OK, I'll go make that happen.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2017-08-07 20:32:22 | Re: pgsql: Record full paths of programs sought by "configure". |
Previous Message | Andrew Dunstan | 2017-08-07 20:15:11 | Re: pgsql: Record full paths of programs sought by "configure". |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Geoghegan | 2017-08-07 20:21:19 | Re: Crash report for some ICU-52 (debian8) COLLATE and work_mem values |
Previous Message | Andrew Dunstan | 2017-08-07 20:15:11 | Re: pgsql: Record full paths of programs sought by "configure". |