On 3 March 2016 at 05:26, Alvaro Herrera <alvhe...@2ndquadrant.com> wrote:
> > Pushed it with that fix. I also added a further "data_" prefix, so it's > "data_${name}_XXXX" now. Hopefully this is less problematic than > yesterday's ... > > On the Perl 5.8.8 test env I've set up now, per http://www.postgresql.org/message-id/camsr+ygr6pu-guyp-ft98xwxasc9n6j-awzaqxvw_+p3rtc...@mail.gmail.com master currently fails with t/004_timeline_switch....."remove_tree" is not exported by the File::Path module remove_tree is only in File::Path 2.07 but Perl 5.8.8 has 1.08. No buildfarm member has complained, so clearly we don't actually test with the stated supported Perl version. The attached patch fixes it to use the legacy File::Path interface 'rmtree' so master runs on 588 again. -- Craig Ringer http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services
From a703cd78cf834df794253bb1cfd897daa96d244a Mon Sep 17 00:00:00 2001 From: Craig Ringer <cr...@2ndquadrant.com> Date: Thu, 3 Mar 2016 13:18:23 +0800 Subject: [PATCH] TAP: File::Path::remove_tree is new in 2.x, use 1.x rmtree instead Preserve Perl 5.8.8 support by using the legacy rmtree call instead of remove_tree from File::Path 2.x. --- src/test/recovery/t/004_timeline_switch.pl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/test/recovery/t/004_timeline_switch.pl b/src/test/recovery/t/004_timeline_switch.pl index 58bf580..a180b94 100644 --- a/src/test/recovery/t/004_timeline_switch.pl +++ b/src/test/recovery/t/004_timeline_switch.pl @@ -3,7 +3,7 @@ # on a new timeline. use strict; use warnings; -use File::Path qw(remove_tree); +use File::Path; use PostgresNode; use TestLib; use Test::More tests => 1; @@ -46,7 +46,7 @@ $node_master->teardown_node; $node_standby_1->promote; # Switch standby 2 to replay from standby 1 -remove_tree($node_standby_2->data_dir . '/recovery.conf'); +File::Path::rmtree($node_standby_2->data_dir . '/recovery.conf'); my $connstr_1 = $node_standby_1->connstr; $node_standby_2->append_conf( 'recovery.conf', qq( -- 2.1.0
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers