Buildfarm animal caiman showing a plperl test issue with newer Perl versions

Started by Alexander Lakhinover 1 year ago3 messages
#1Alexander Lakhin
exclusion@gmail.com
#2Andrew Dunstan
andrew@dunslane.net
In reply to: Alexander Lakhin (#1)
Re: Buildfarm animal caiman showing a plperl test issue with newer Perl versions

On 2024-06-24 Mo 12:00 AM, Alexander Lakhin wrote:

Hello hackers,

As recent caiman failures ([1], [2], ...) show, plperl.sql is
incompatible
with Perl 5.40. (The last successful test runs took place when cayman
had Perl 5.38.2 installed: [3].)

FWIW, I've found an already-existing fix for the issue [4] and a note
describing the change for Perl 5.39.10 [5].

[1]
https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=caiman&dt=2024-06-24%2001%3A34%3A23
[2]
https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=caiman&dt=2024-06-24%2000%3A15%3A16
[3]
https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=caiman&dt=2024-05-02%2021%3A57%3A17
[4]
https://git.alpinelinux.org/aports/tree/community/postgresql14/fix-test-plperl-5.8-pragma.patch?id=28aeb872811f59a7f646aa29ed7c9dc30e698e65
[5]
https://metacpan.org/release/PEVANS/perl-5.39.10/changes#Selected-Bug-Fixes

It's a very odd bug. I guess we should just backpatch the removal of
that redundant version check in plc_perlboot.pl, probably all the way
down to 9.2 since godwit builds and tests with plperl that far back, and
some day in the not too distant future it will upgrade to perl 5.40.

cheers

andrew

--
Andrew Dunstan
EDB: https://www.enterprisedb.com

#3Andrew Dunstan
andrew@dunslane.net
In reply to: Andrew Dunstan (#2)
Re: Buildfarm animal caiman showing a plperl test issue with newer Perl versions

On 2024-06-24 Mo 6:46 AM, Andrew Dunstan wrote:

On 2024-06-24 Mo 12:00 AM, Alexander Lakhin wrote:

Hello hackers,

As recent caiman failures ([1], [2], ...) show, plperl.sql is
incompatible
with Perl 5.40. (The last successful test runs took place when cayman
had Perl 5.38.2 installed: [3].)

FWIW, I've found an already-existing fix for the issue [4] and a note
describing the change for Perl 5.39.10 [5].

[1]
https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=caiman&dt=2024-06-24%2001%3A34%3A23
[2]
https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=caiman&dt=2024-06-24%2000%3A15%3A16
[3]
https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=caiman&dt=2024-05-02%2021%3A57%3A17
[4]
https://git.alpinelinux.org/aports/tree/community/postgresql14/fix-test-plperl-5.8-pragma.patch?id=28aeb872811f59a7f646aa29ed7c9dc30e698e65
[5]
https://metacpan.org/release/PEVANS/perl-5.39.10/changes#Selected-Bug-Fixes

It's a very odd bug. I guess we should just backpatch the removal of
that redundant version check in plc_perlboot.pl, probably all the way
down to 9.2 since godwit builds and tests with plperl that far back,
and some day in the not too distant future it will upgrade to perl 5.40.

Done.

cheers

andrew

--
Andrew Dunstan
EDB: https://www.enterprisedb.com