pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Robert Haas <rhaas@postgresql.org>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Minor fixes to pg_combinebackup and its documentation.
Date: Fri, 26 Apr 2024 12:49:44 +0000
Message-ID: <E1s0L1L-0008ZP-ID@gemulon.postgresql.org> (raw)

Minor fixes to pg_combinebackup and its documentation.

The --tablespace-mapping option was specified with required_argument
rather than no_argument, which is wrong. Since the actual argument
string passed to getopt_long() included "T:", the single-character
form of the option still worked, but the long form did not. Repair.

The call to getopt_long() erroneously included "P", which doesn't
correspond to any supported option. Remove.

The help message used "do not" in one place and "don't" in another.
Standardize on "do not".

The documentation erroneously stated that the tablespace mappings
would be applied relative to the pathnames in the first backup
specified on the command line, rather than the final one. Fix.

Thanks to Tomas Vondra and Daniel Gustafsson for alerting me to
these mistakes.

Discussion: http://postgr.es/m/CA+TgmoYFznwwaZhHSF1Ze7JeyBv-1yOoSrucKMw37WpF=7RP8g@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/1713e3d6cd393fcc1d4873e75c7fa1f6c7023d75

Modified Files
--------------
doc/src/sgml/ref/pg_combinebackup.sgml      | 2 +-
src/bin/pg_combinebackup/pg_combinebackup.c | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)



view thread (2+ messages)

Message-ID: <E1s0L1L-0008ZP-ID@gemulon.postgresql.org>
Permalink:  ../E1s0L1L-0008ZP-ID@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1s0L1L-0008ZP-ID@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: rhaas@postgresql.org, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Minor fixes to pg_combinebackup and its documentation.
  In-Reply-To: <E1s0L1L-0008ZP-ID@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox