public inbox for [email protected]  

help / color / mirror / Atom feed

From: Mikael Kjellström <[email protected]>
To: Andrew Dunstan <[email protected]>
To: PostgreSQL-development <[email protected]>
To: [email protected]
Subject: Re: [buildfarm-members] BuildFarm client release 4.19
Date: Mon, 1 May 2017 21:19:44 +0200
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgsql-hackers>


On 2017-05-01 21:10, Andrew Dunstan wrote:
> Not sure I understand what "rerun a branch" from scratch means. If you
> zap the branch directory you lose all its state. That's generally a bad
> thing.

I mean like the first time when you set up the buildfarm client / 
branch.  And it's not something I recomend doing all the time but some 
times when the buildfarm client doesn't clean up after it self and there 
are a lot of crap left it's "easier" to just zap everything and start 
over to get it running again.


> Anyway, this patch should fix it for all uses. It creates the directory
> if it doesn't exist.
>
>     diff --git a/PGBuild/Utils.pm b/PGBuild/Utils.pm
>     index 91c1362..175eaa7 100644
>     --- a/PGBuild/Utils.pm
>     +++ b/PGBuild/Utils.pm
>     @@ -14,6 +14,8 @@ See accompanying License file for license details
>      use strict;
>      use warnings;
>
>     +use File::Path;
>     +
>      use Exporter   ();
>      our (@ISA, @EXPORT, @EXPORT_OK, %EXPORT_TAGS);
>
>     @@ -30,8 +32,9 @@ use vars qw($VERSION); $VERSION = 'REL_4.19';
>      sub run_log
>      {
>          my $command = shift;
>     -    my $file=
>     -
>     "$main::branch_root/$main::st_prefix$main::logdirname/lastcomand.log";
>     +   my $filedir =
>     "$main::branch_root/$main::st_prefix$main::logdirname";
>     +   mkpath($filedir);
>     +   my $file= "$filedir/lastcomand.log";
>          unlink $file;
>          system("$command > $file 2>&1");
>          my @loglines;

Ok.

Thanks.  Will try it out.

/Mikael


-- 
Sent via pgsql-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

view thread (15+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: [buildfarm-members] BuildFarm client release 4.19
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox