Re: plperl: update ppport.h and fix configure version check

2021-10-07 Thread Tom Lane
I wrote: > * adjust configure and docs to set 5.8.3 as the minimum perl version When I went to update the docs, I found they already said 5.8.3 is the minimum. Excavating in the git log led me to this old discussion: https://www.postgresql.org/message-id/flat/16894.1501392088%40sss.pgh.pa.us#2c7

Re: plperl: update ppport.h and fix configure version check

2021-10-07 Thread Tom Lane
AFAICS we have consensus on doing these things (in HEAD only): * update ppport.h to perl 5.34.0 * adjust configure and docs to set 5.8.3 as the minimum perl version * adjust docs to say we don't test or support AIX below 7.1. I'll go make these things happen. regards, t

Re: plperl: update ppport.h and fix configure version check

2021-10-06 Thread Andrew Dunstan
On 10/5/21 10:30 AM, Tom Lane wrote: > Andrew Dunstan writes: >> Seems OK. Note that the Msys DTK perl currawong uses to build with is >> ancient (5.6.1). That's going to stay as it is until it goes completely >> out of scope in about 13 months. The perl it builds plperl against is >> much more

Re: plperl: update ppport.h and fix configure version check

2021-10-05 Thread Tom Lane
Andrew Dunstan writes: > Seems OK. Note that the Msys DTK perl currawong uses to build with is > ancient (5.6.1). That's going to stay as it is until it goes completely > out of scope in about 13 months. The perl it builds plperl against is > much more modern - 5.16.3. That brings up something I

Re: plperl: update ppport.h and fix configure version check

2021-10-05 Thread Andrew Dunstan
On 10/4/21 11:12 PM, Tom Lane wrote: > > In short: (a) we're not testing against anything older than 5.8.3 > and (b) it seems quite unlikely that anybody cares about 5.8.x anyway. > So if we want to mess with this, maybe we should set the cutoff > to 5.8.3 not 5.8.1. > > Seems OK. Note t

Re: plperl: update ppport.h and fix configure version check

2021-10-05 Thread Tom Lane
=?utf-8?Q?Dagfinn_Ilmari_Manns=C3=A5ker?= writes: > Tom Lane writes: >> Hard to tell if that is a local update or official IBM distribution. > Looks like they update the Perl version in OS updates and service packs: > https://www.ibm.com/support/pages/aix-perl-updates-and-support-perlrte Oh, in

Re: plperl: update ppport.h and fix configure version check

2021-10-05 Thread Dagfinn Ilmari Mannsåker
Tom Lane writes: > Interestingly, although cpan's table says AIX 7.1 shipped with perl > 5.10.1, what's actually on those buildfarm animals is > > tgl@gcc111:[/home/tgl]which perl > /usr/bin/perl > tgl@gcc111:[/home/tgl]ls -l /usr/bin/perl > lrwxrwxrwx1 root system 29 Nov 09 202

Re: plperl: update ppport.h and fix configure version check

2021-10-05 Thread Tom Lane
=?utf-8?Q?Dagfinn_Ilmari_Manns=C3=A5ker?= writes: > Daniel Gustafsson writes: >> Not being able to test against older versions in the builfarm seems like a >> pretty compelling reason to set 5.8.3 as the required version. > Looking at the list of Perl versions shipped with various OSes > (https:

Re: plperl: update ppport.h and fix configure version check

2021-10-05 Thread Dagfinn Ilmari Mannsåker
Daniel Gustafsson writes: >> On 5 Oct 2021, at 05:12, Tom Lane wrote: > >> In short: (a) we're not testing against anything older than 5.8.3 >> and (b) it seems quite unlikely that anybody cares about 5.8.x anyway. >> So if we want to mess with this, maybe we should set the cutoff >> to 5.8.3 no

Re: plperl: update ppport.h and fix configure version check

2021-10-05 Thread Daniel Gustafsson
> On 5 Oct 2021, at 05:12, Tom Lane wrote: > In short: (a) we're not testing against anything older than 5.8.3 > and (b) it seems quite unlikely that anybody cares about 5.8.x anyway. > So if we want to mess with this, maybe we should set the cutoff > to 5.8.3 not 5.8.1. Not being able to test a

Re: plperl: update ppport.h and fix configure version check

2021-10-04 Thread Tom Lane
=?utf-8?Q?Dagfinn_Ilmari_Manns=C3=A5ker?= writes: > Andres' recent post¹ about PL/Perl on Windows and linked-to² patch > containing an erroneous version check made me realise that we haven't > updated our copy of ppport.h since 2009. Attached is a patch that does > that, and applies code changes