| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> | 
|---|---|
| To: | Andrew Dunstan <andrew(at)dunslane(dot)net> | 
| Cc: | Josh Berkus <josh(at)agliodbs(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Tim Bunce <Tim(dot)Bunce(at)pobox(dot)com> | 
| Subject: | Re: PL/perl should fail on configure, not make | 
| Date: | 2013-01-09 21:12:40 | 
| Message-ID: | 2211.1357765960@sss.pgh.pa.us | 
| Views: | Whole Thread | Raw Message | Download mbox | Resend email | 
| Thread: | |
| Lists: | pgsql-hackers | 
Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
> On 01/09/2013 10:16 AM, Tom Lane wrote:
>> Actually, if we were to try to clean this up, I'd suggest moving that
>> logic into the configure script --- it's not apparent to me why it's
>> a good idea to be changing configure-determined values in the Makefile.
>> But in any case this would have to be done by somebody who's in a
>> position to test on affected platforms.
> Here's a patch which does that and produces configure traces like this 
> on Mingw:
>     checking for Perl archlibexp... C:/Perl/lib
>     checking for Perl privlibexp... C:/Perl/lib
>     checking for Perl useshrplib... true
>     checking for flags to link embedded Perl... -LC:/Perl/lib/CORE -lperl512
> which seems to be what we want.
> Given that, you should be able to write a reasonably portable configure 
> test for library presence.
Looks good.  If you're happy with that then I can undertake to add a
libperl.so probe based on AC_TRY_LINK with the unmodified value of
$perl_embed_ldflags.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Simon Riggs | 2013-01-09 21:15:16 | Re: Reducing size of WAL record headers | 
| Previous Message | Simon Riggs | 2013-01-09 21:08:55 | Re: Index build temp files |