From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | Re: pgsql: Make PostgreSQL::Test::Cluster compatible with all live branches |
Date: | 2022-03-31 18:18:33 |
Message-ID: | 534ed6b2-6f25-4b0d-4159-2a7f09b61d4b@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
On 3/31/22 12:50, Tom Lane wrote:
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>> There are three ways out of this that I can see:
>> . carry a copy of parent.pm in src/test/perl (It's very small)
>> . use the older and heavier 'base' module which goes back to 5.004, and
>> does much the same thing (and a lot more)
>> . just do directly what parent.pm's import() does, as in the attached,
>> which I have tested down to version 10.
>> On the whole I think I prefer the last.
> I like #3 too, given that it's such a small patch. I've verified
> that it allows regression tests to run on perl 5.8.8, which is the
> oldest I have handy other than prairiedog's 5.8.3 (but that machine
> is tied up and I don't feel like trying to test this concurrently
> with a buildfarm run).
>
>
I think it's safe enough. Pushed.
cheers
andrew
--
Andrew Dunstan
EDB: https://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2022-03-31 18:21:17 | pgsql: Print information about type of test and subdirectory before run |
Previous Message | Andrew Dunstan | 2022-03-31 18:17:26 | pgsql: Remove use of perl parent module in Cluster.pm |