| 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 10:09:34 |
| Message-ID: | 230c42b8-5758-3616-b4fd-df9191f0c273@dunslane.net |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers |
On 3/30/22 21:12, Tom Lane wrote:
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>>> On Mar 30, 2022, at 8:19 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>>> I think this means that its old Perl version misinterprets
>>> use parent -norequire, qw(PostgreSQL::Test::Cluster);
>>> as a request to include "parent.pm". Is there a more
>>> backwards-compatible way to spell that?
>> Not that I know of. Googling tells me it was a core module from 5.10.1. I can revert tomorrow if necessary :-(
I must have been half asleep when I posted this.
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.
cheers
andrew
--
Andrew Dunstan
EDB: https://www.enterprisedb.com
| Attachment | Content-Type | Size |
|---|---|---|
| cluster-parent-fix.patch | text/x-patch | 1.1 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Daniel Gustafsson | 2022-03-31 10:12:34 | pgsql: doc: Fix typo in ANALYZE documentation |
| Previous Message | Daniel Gustafsson | 2022-03-31 09:26:04 | pgsql: Add diagnostic output on error in pump_until |