Re: pgsql: plperl: windows: Use Perl_setlocale on 5.28+, fixing compile fai

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: pgsql-committers(at)lists(dot)postgresql(dot)org, Dagfinn Ilmari Mannsåker <ilmari(at)ilmari(dot)org>
Subject: Re: pgsql: plperl: windows: Use Perl_setlocale on 5.28+, fixing compile fai
Date: 2022-02-02 03:07:53
Message-ID: 1052579.1643771273@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Andres Freund <andres(at)anarazel(dot)de> writes:
> On 2022-01-31 15:08:30 -0500, Tom Lane wrote:
>> Done now.

> Thanks!

Just for the archives' sake --- seems this broke pademelon [1]:

cc -Ae -g +O0 -Wp,-H16384 +Z -I. -I. -I../../../src/include -D_XOPEN_SOURCE_EXTENDED -I/usr/local/libxml2-2.6.23/include/libxml2 -I/usr/local/include -DDEBUGGING -I/opt/perl5.8.9/lib/5.8.9/PA-RISC2.0-multi/CORE -c -o plperl.o plperl.c
...
cc: "plperl.c", line 669: error 1000: Unexpected symbol: "do".
cc: "plperl.c", line 669: error 1000: Unexpected symbol: "_sv".
cc: error 2017: Cannot recover from earlier errors, terminating.
make[3]: *** [plperl.o] Error 1

I've not tried to trace that in detail, but it seems clear that
something in the newer ppport.h is assuming a C99-compliant
compiler. I can hardly blame the Perl guys for assuming that
in 2022, so I'm just going to remove --with-perl from that
animal.

(It appears that Python 3 assumes C99-compliance too, in
a different way [2], so I've also had to remove --with-python.
Maybe I could resurrect that if I built Python 2.7 on that
machine, but I don't think I'm going to summon the motivation.
The world has clearly Moved On from pre-C99 compilers.)

regards, tom lane

[1] https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=pademelon&dt=2022-02-02%2000%3A47%3A30
[2] https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=pademelon&dt=2022-02-02%2000%3A21%3A18

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Thomas Munro 2022-02-02 03:13:28 pgsql: Fix recovery conflict in 027_stream_regress.pl.
Previous Message Tom Lane 2022-02-02 00:03:59 pgsql: Replace use of deprecated Python module distutils.sysconfig, tak