From: Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
Subject: Re: Announcing Release 6 of PostgreSQL Buildfarm client
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: buildfarm-members@postgresql.org
References: <59bfeba7-8ee0-31da-9db9-610002f9c841@2ndQuadrant.com>
 <8725.1515087312@sss.pgh.pa.us> <9420.1515088313@sss.pgh.pa.us>
Message-ID: <0b3217e4-b275-ba4d-0571-30b4aae237e0@2ndQuadrant.com>
Date: Thu, 4 Jan 2018 15:39:41 -0500
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
 Thunderbird/52.4.0
MIME-Version: 1.0
In-Reply-To: <9420.1515088313@sss.pgh.pa.us>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Content-Language: en-MW
Precedence: bulk


On 01/04/2018 12:51 PM, Tom Lane wrote:
> I wrote:
>> which means I get mail every time the cron script runs.  Needless
>> to say, that will not do.  Previously, mail only showed up when
>> there was a git checkout failure.  How do I get back to the
>> old behavior?
> Diff'ing the v5 and v6 releases soon provided the answer: for some
> reason v6 is forcing the verbose switch on.  I hope that was
> unintentional?  Anyway, I'm happy again after applying this patch:
>
> --- PGBuild/Options.pm~ 2018-01-04 08:38:29.000000000 -0500
> +++ PGBuild/Options.pm  2018-01-04 12:40:14.000000000 -0500
> @@ -78,8 +78,6 @@
>      GetOptions(%standard_options, @_)
>        || die "bad command line";
>  
> -    # override GetOptions default for :i
> -    $verbose = 1 if (defined($verbose) && $verbose==0);
>      $verbose ||= 0; # stop complaints about undefined var in numeric comparison
>  }
>  


This should fix it:
<https://raw.githubusercontent.com/PGBuildFarm/client-code/5c71fb6ba1be83db6aca45e718b4a541b0b89f4c/PGBuild/Options.pm>

cheers

andrew

-- 
Andrew Dunstan                https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services