Re: PATCH: Configurable file mode mask - Mailing list pgsql-hackers

From Stephen Frost
Subject Re: PATCH: Configurable file mode mask
Date
Msg-id 20180407222953.GB27724@tamriel.snowman.net
Whole thread Raw
In response to Re: PATCH: Configurable file mode mask  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Greetings,

* Tom Lane (tgl@sss.pgh.pa.us) wrote:
> Stephen Frost <sfrost@snowman.net> writes:
> > * Tom Lane (tgl@sss.pgh.pa.us) wrote:
> >> https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=culicidae&dt=2018-04-07%2021%3A50%3A02
>
> > Yes, I'm taking a look at it.
>
> Since other animals are coming back successfully, my first suspicion
> lights on culicidae's use of EXEC_BACKEND.  Did you test that case?
> (If not, this bodes ill for the Windows results.)

The vast majority of this patch isn't something that's relevant to
Windows anyway, so I'm hopeful that those animals will stay green (if
not, it's probably that we need to mark some other test as
not-on-Windows).

We have EXEC_BACKEND on the Unix-y systems too though, so that needs to
be fixed.  I'm pretty sure the issue here is that SubPostmasterMain()
needs to also be checking/updating the umask() based on the data
directory, as is done in PostmasterMain.

Testing that out now and if that looks good then I'll push that and
hopefully it'll make cullcidae green.

Thanks!

Stephen

Attachment

pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Searching for: Fast windows buildfarm animal
Next
From: Stephen Frost
Date:
Subject: Re: PATCH: Configurable file mode mask