Re: [HACKERS] Old versions of Test::More

On 04/21/2017 09:22 PM, Craig Ringer wrote:
>
>
> On 22 Apr. 2017 4:23 am, "Tom Lane" <tgl@sss.pgh.pa.us
> <mailto:tgl@sss.pgh.pa.us>> wrote:
>
>     Peter Eisentraut <peter.eisentraut@2ndquadrant.com
>     <mailto:peter.eisentraut@2ndquadrant.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