Date: Mon, 20 Sep 2021 22:11:53 -0700
From: Andres Freund <andres@anarazel.de>
To: Michael Paquier <michael@paquier.xyz>
Cc: pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: Support "postgres -C" with runtime-computed GUCs
Message-ID: <20210921051153.slo7phuc2ddqzqn4@alap3.anarazel.de>
References: <E1mQglq-0007xB-Jw@gemulon.postgresql.org>
 <20210921032040.lyl4lcax37aedx2x@alap3.anarazel.de>
 <YUlVzPkVgEPjtmIB@paquier.xyz>
 <YUliupGihilpM2Xm@paquier.xyz>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <YUliupGihilpM2Xm@paquier.xyz>
Archived-At: 
 <https://www.postgresql.org/message-id/20210921051153.slo7phuc2ddqzqn4%40alap3.anarazel.de>
Precedence: bulk

Hi,

On 2021-09-21 13:42:34 +0900, Michael Paquier wrote:
> On Tue, Sep 21, 2021 at 12:47:24PM +0900, Michael Paquier wrote:
> > This would print the parameter, while getting to stderr mostly a
> > "could not start server".  Would that work for you?
> 
> And..  Here is an actual patch.

Cool!


> The first test works here on Win32, but I don't have a CI environment
> with an unprivileged account.

I assume it's just a typo, but the problem is the opposite, it's a
*privileged* CI account causing trouble...


> Could you check if the first case would work in your environment?  If
> not, I would not mind just removing the test.  I have discovered a
> separate bug thanks to it, but that's not worth breaking your case
> either.

It's running now: https://cirrus-ci.com/build/5927040688848896

If interesting I can also come up with the necessary steps to activate
it in your repo...

Greetings,

Andres Freund