Re: Still another race condition in recovery TAP tests
Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
From: Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
To: Noah Misch <noah@leadboat.com>, Craig Ringer <craig@2ndquadrant.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>,
PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Date: 2017-10-13T12:42:05Z
Lists: pgsql-hackers
On 10/13/2017 01:04 AM, Noah Misch wrote: > On Fri, Oct 06, 2017 at 05:57:24PM +0800, Craig Ringer wrote: >> On 6 October 2017 at 14:03, Noah Misch <noah@leadboat.com> wrote: >>> On Fri, Sep 08, 2017 at 10:32:03PM -0400, Tom Lane wrote: >>>> (I do kinda wonder why we rolled our own RecursiveCopy; surely there's >>>> a better implementation in CPAN?) >>> Fewer people will test as we grow the list of modules they must first install. >> Meh, I don't buy that. At worst, all we have to do is provide a script >> that fetches them, from distro repos if possible, and failing that >> from CPAN. >> >> With cpanminus, that's pretty darn simple too. > If the tree had such a script and it were reliable, then yes, it would matter > little whether the script procured one module or five. > > Not everyone has cpanminus installed either. My approach in the buildfarm code is to lean over backwards in order to avoid non-standard modules. For the problem at hand we use cp/xcopy, but the tree being copied is stable so we don't run into the disappearing/changing file problem. cheers andrew -- Andrew Dunstan https://www.2ndQuadrant.com PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Commits
-
Fix RecursiveCopy.pm to cope with disappearing files.
- e183530550dc 11.0 landed
- bd18960cb93f 10.0 landed
- 64e2b29bdea0 9.6.6 landed
-
Refactor Perl test code
- 1caef31d9e55 9.6.0 cited