Re: pgsql: Fix EXEC BACKEND + Windows builds for group privs - Mailing list pgsql-committers

From Stephen Frost
Subject Re: pgsql: Fix EXEC BACKEND + Windows builds for group privs
Date
Msg-id 20180409025231.GH27724@tamriel.snowman.net
Whole thread Raw
In response to Re: pgsql: Fix EXEC BACKEND + Windows builds for group privs  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-committers
Greetings,

* Tom Lane (tgl@sss.pgh.pa.us) wrote:
> Stephen Frost <sfrost@snowman.net> writes:
> > Fix EXEC BACKEND + Windows builds for group privs
>
> jacana seems to think there's still an issue here:
>
> https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=jacana&dt=2018-04-09%2000%3A00%3A41
>
> It's pretty much the same symptom:
>
> Apr 08 20:58:22 Running this script will delete the old cluster's data files:
> Apr 08 20:58:22     delete_old_cluster.bat
> Apr 08 20:58:33 ++ find /home/pgrunner/bf/root/HEAD/pgsql.build/src/bin/pg_upgrade/tmp_check/data -type f '!' -perm
640
> Apr 08 20:58:33 ++ wc -l
> Apr 08 20:58:42 + '[' 3361 -ne 0 ']'
> Apr 08 20:58:42 + echo 'files in PGDATA with permission != 640'
> Apr 08 20:58:42 files in PGDATA with permission != 640
> Apr 08 20:58:42 + exit 1
> Apr 08 20:58:42 make: *** [check] Error 1

Yes, I was just looking at that and working up what I believe will be a
fix.  This is exactly the kind of test I was anticipating getting run
unintentionally on Windows, sorry about that, but I don't have a Windows
test environment currently.

I've pushed what I believe will fix this, which is essentially "don't
try to run this test on this platform."  Tested it locally (under Linux)
and it didn't blow up, so hopefully this doesn't make things worse, in
any case.

If there's more that's needed to fix jacana, I'll see about dealing with
it first thing in the morning.

Thanks!

Stephen

Attachment

pgsql-committers by date:

Previous
From: Stephen Frost
Date:
Subject: pgsql: Skip permissions test under MINGW/Windows
Next
From: David Rowley
Date:
Subject: Re: pgsql: Support partition pruning at execution time