pg_basebackup has an accidentaly separated help message

Started by Kyotaro Horiguchiover 2 years ago6 messageshackers
Jump to latest
#1Kyotaro Horiguchi
horikyota.ntt@gmail.com

Hello.

pg_basebackup.c: got the following message lines:

printf(_(" -i, --incremental=OLDMANIFEST\n"));
printf(_(" take incremental backup\n"));

I'd suggest merging these lines as follows (and the attached patch).

+	printf(_("  -i, --incremental=OLDMANIFEST\n"
+			 "                         take incremental backup\n"));

regards.

--
Kyotaro Horiguchi
NTT Open Source Software Center

Attachments:

pg_basebackup_merge_help.difftext/x-patch; charset=us-asciiDownload+2-2
#2Kyotaro Horiguchi
horikyota.ntt@gmail.com
In reply to: Kyotaro Horiguchi (#1)
Re: pg_basebackup has an accidentaly separated help message

printf(_(" -i, --incremental=OLDMANIFEST\n"));
printf(_(" take incremental backup\n"));

I'd suggest merging these lines as follows (and the attached patch).

+	printf(_("  -i, --incremental=OLDMANIFEST\n"
+			 "                         take incremental backup\n"));

Sorry, but I found another instance of this.

printf(_(" -T, --tablespace-mapping=OLDDIR=NEWDIR\n"));
printf(_(" relocate tablespace in OLDDIR to NEWDIR\n"));

The attached patch contains both of the above fixes.

regards.

--
Kyotaro Horiguchi
NTT Open Source Software Center

Attachments:

pg_basebackup_merge_help.difftext/x-patch; charset=us-asciiDownload+4-4
#3Michael Paquier
michael@paquier.xyz
In reply to: Kyotaro Horiguchi (#2)
Re: pg_basebackup has an accidentaly separated help message

On Mon, Dec 25, 2023 at 02:39:16PM +0900, Kyotaro Horiguchi wrote:

The attached patch contains both of the above fixes.

Good catches, let's fix them. You have noticed that while translating
these new messages, I guess?
--
Michael

#4Kyotaro Horiguchi
horikyota.ntt@gmail.com
In reply to: Michael Paquier (#3)
Re: pg_basebackup has an accidentaly separated help message

At Mon, 25 Dec 2023 15:42:41 +0900, Michael Paquier <michael@paquier.xyz> wrote in

On Mon, Dec 25, 2023 at 02:39:16PM +0900, Kyotaro Horiguchi wrote:

The attached patch contains both of the above fixes.

Good catches, let's fix them. You have noticed that while translating
these new messages, I guess?

Yes. So, it turns out that they're found after they have been
committed.

Because handling a large volume of translations all at once is
daunting, I am maintaining translations locally to avoid that.

regards.

--
Kyotaro Horiguchi
NTT Open Source Software Center

#5Michael Paquier
michael@paquier.xyz
In reply to: Kyotaro Horiguchi (#4)
Re: pg_basebackup has an accidentaly separated help message

On Mon, Dec 25, 2023 at 05:07:28PM +0900, Kyotaro Horiguchi wrote:

Yes. So, it turns out that they're found after they have been
committed.

No problem. I've just applied what you had. I hope this makes your
life a bit easier ;)
--
Michael

#6Kyotaro Horiguchi
horikyota.ntt@gmail.com
In reply to: Michael Paquier (#5)
Re: pg_basebackup has an accidentaly separated help message

At Tue, 26 Dec 2023 19:04:53 +0900, Michael Paquier <michael@paquier.xyz> wrote in

On Mon, Dec 25, 2023 at 05:07:28PM +0900, Kyotaro Horiguchi wrote:

Yes. So, it turns out that they're found after they have been
committed.

No problem. I've just applied what you had. I hope this makes your
life a bit easier ;)

Thanks for committing this!

regards.

--
Kyotaro Horiguchi
NTT Open Source Software Center