Re: buildfarm's typedefs list has gone completely nutso

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: Andres Freund <andres(at)anarazel(dot)de>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: buildfarm's typedefs list has gone completely nutso
Date: 2019-07-18 15:42:12
Message-ID: 24705.1563464532@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

I wrote:
> ... I just realized from looking at its
> config that you have it set to do so at most twice a week:

> 'dow' => [
> 1,
> 4
> ]

I was still confused, seeing that today is Thursday, as to why
komodoensis didn't update its typedefs list in the run it just
finished. Looking at the buildfarm script (in sub
check_optional_step), it seems the "dow" filter is implemented
like this:

return
if (exists $oconf->{dow}
&& grep { $_ eq $wday } @{ $oconf->{dow} });

I'm the world's worst Perl programmer, but isn't that backwards?
It seems like it will return undef if today matches any entry
of the dow list, making dow a blacklist of weekdays *not* to run
the step on. That's not what I would have expected it to mean,
although build-farm.conf.sample is surely unclear on the point.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Daniel Westermann (DWE) 2019-07-18 15:46:00 Re: Fw: Documentation fix for adding a column with a default value
Previous Message Tom Lane 2019-07-18 15:27:49 Re: buildfarm's typedefs list has gone completely nutso