Horton

Re: pgsql: Adjust configure to insist on Perl version >= 5.8.3.

Tom Lane <tgl@sss.pgh.pa.us>

View thread View on postgr.es
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Andrew Dunstan <andrew@dunslane.net>
Cc: Noah Misch <noah@leadboat.com>, Daniel Gustafsson <daniel@yesql.se>, Postgres hackers <pgsql-hackers@lists.postgresql.org>, Craig Ringer <craig.ringer@enterprisedb.com>
Date: 2021-10-11T15:03:35Z
Lists: pgsql-hackers 
Andrew Dunstan <andrew@dunslane.net> writes:
> On 10/9/21 10:25 PM, Noah Misch wrote:
>> You mentioned prairiedog uses IPC::Run 0.79.  That's from 2005.  (Perl 5.8.3
>> is from 2004, and Test::More 0.87 is from 2009.)  I'd just use 0.79 in the
>> README recipe.  IPC::Run is easy to upgrade, so if we find cause to rely on a
>> newer version, I'd be fine updating that requirement.

> Why don't we specify the minimum versions required of these somewhere in
> the perl code? Perl is pretty good at this.

configure already checks Test::More's version.  I proposed downthread
that it should also check IPC::Run, but didn't pull that trigger yet.

			regards, tom lane

Commits

  1. Make configure check for minimum required version of IPC::Run.

  2. Doc: update testing recipe in src/test/perl/README.

  3. Update test/perl/README to insist on Perl version >= 5.8.3, too.

  4. Adjust configure to insist on Perl version >= 5.8.3.

  5. Doc: specify that the minimum supported version of Perl is 5.8.3.