From: | Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com> |
---|---|
To: | Craig Ringer <craig(dot)ringer(at)2ndquadrant(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
Subject: | Re: Old versions of Test::More |
Date: | 2017-04-22 14:29:10 |
Message-ID: | e994927e-8e2d-477b-18e7-2fd18c8e4cd2@2ndQuadrant.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 04/21/2017 09:22 PM, Craig Ringer wrote:
>
>
> On 22 Apr. 2017 4:23 am, "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us
> <mailto:tgl(at)sss(dot)pgh(dot)pa(dot)us>> wrote:
>
> Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com
> <mailto:peter(dot)eisentraut(at)2ndquadrant(dot)com>> writes:
> > On 4/21/17 14:49, Andrew Dunstan wrote:
> >> I'll add a comment, but doing it in PostgresNode.pm would mean
> jacana
> >> (for instance) couldn't run any of the TAP tests. I'mm looking at
> >> installing a sufficiently modern Test::Simple package (includes
> >> Test::More and test::Build) there, but other oldish machines
> could also
> >> be affected.
>
> > Or you could define note() as an empty function if it doesn't exist.
>
> +1. I'm really not at all happy with the prospect that every time
> somebody adds a use of "note" to some new TAP test, we're going to
> get a complaint later that that test no longer works on jacana.
> We need to either decide that non-ancient Test::More is a hard
> requirement for all the tests
>
>
> That seems like a no-brainer TBH. Why are we bothering with backwards
> compat with ancient versions of test frameworks? It seems like a
> colossal waste of time for no benefit.
>
OK, I have pushed a requirement for a minimum version of Test::More into
TestLib.pm, a better place for it than PostgresNode.pm as not all tests
use the latter.
jacana has been upgraded to use a sufficiently modern Test::More.
cheers
andrew
--
Andrew Dunstan https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2017-04-22 14:31:58 | Re: Small patch for pg_basebackup argument parsing |
Previous Message | Greg Stark | 2017-04-22 14:12:30 | Re: multithreading in Batch/pipelining mode for libpq |