Re: Postgres and --config-file option

From: Peter Eisentraut <peter(at)eisentraut(dot)org>
To: "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>, Aleksander Alekseev <aleksander(at)timescale(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Nathan Bossart <nathandbossart(at)gmail(dot)com>
Subject: Re: Postgres and --config-file option
Date: 2024-02-07 09:58:40
Message-ID: 0ffab2f8-496b-463c-88f4-f9ca05fb5371@eisentraut.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 02.02.24 22:27, David G. Johnston wrote:
> On Fri, Feb 2, 2024 at 2:23 PM David G. Johnston
> <david(dot)g(dot)johnston(at)gmail(dot)com <mailto:david(dot)g(dot)johnston(at)gmail(dot)com>> wrote:
>
> On Mon, Jan 15, 2024 at 4:35 AM Aleksander Alekseev
> <aleksander(at)timescale(dot)com <mailto:aleksander(at)timescale(dot)com>> wrote:
>
> PFA the patch. It's short but I think it mitigates the problem.
>
>
> I took a look at where these options are discussed in the
> documentation and now feel that we should make these options clear
> more broadly (config and libpq, plus pointing to --name from -c in a
> couple of places).  It doesn't add much verbosity and, frankly, if I
> was to pick one "--name=value" would win and so I'd rather document
> it, leaving -c alone for historical reasons.
>
> I've attached a replacement patch with the additional changes.
>
>
> And I just saw one more apparently undocumented requirement (or a typo)
>
> You must specify the --config-file
>
> The actual parameter is "config_file", so apparently we are supposed to
> either convert underscores to hyphens or we have a typo.

We convert '-' to '_' when parsing long options (see ParseLongOption()
in guc.c). So writing the long options with hyphens should generally be
preferred in documentation.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2024-02-07 10:13:14 Re: Popcount optimization using AVX512
Previous Message Amit Kapila 2024-02-07 09:55:48 Re: Why is subscription/t/031_column_list.pl failing so much?