Re: BUG #19042: Option --help not recognized at the end of command line in pg_restore

From: Daniel Gustafsson <daniel(at)yesql(dot)se>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Peter Eisentraut <peter(at)eisentraut(dot)org>, Nathan Bossart <nathandbossart(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Matthias Hörmann <matthias(dot)hoermann(at)saltation(dot)com>, pgsql-bugs(at)lists(dot)postgresql(dot)org
Subject: Re: BUG #19042: Option --help not recognized at the end of command line in pg_restore
Date: 2025-10-29 20:19:29
Message-ID: 9C49F8A5-CBF8-44E2-915D-62609F90C159@yesql.se
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

> On 29 Oct 2025, at 17:54, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> Peter Eisentraut <peter(at)eisentraut(dot)org> writes:
>>> On 25.10.25 00:26, Daniel Gustafsson wrote:
>>> While hacking on this it seemed like a good idea to move to using progname
>>> consistently, and reduce scope in some places from global variables, but
>>> looking at it now I'm less convinced. At the very least it should be ripped
>>> out into its own patch but for now those changes are left in there.
>
>> The --version outputs intentionally do not use progname, so that you get
>> a proper identification of the program and version independent of what
>> the binary might be called. Please don't change that.

All callsites using handle_help_version_opts were already using progname for —version, this just made it consistent. That being said, I can change it to be consistent in the other direction.

> Hmm, that's important institutional knowledge that should be captured
> somewhere.

Maybe a README in src/bin?

./daniel

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Michael Paquier 2025-10-29 23:36:43 Re: BUG #19093: Behavioral change in walreceiver termination between PostgreSQL 14.17 and 14.18
Previous Message PG Bug reporting form 2025-10-29 20:00:02 BUG #19099: Conditional DELETE from partitioned table with non-updatable partition raises internal error