# [PATCH] i18n: fix command template placeholder format

5 messages from 2022-11-26 to 2022-11-28. Participants: Jean-Noël Avila via GitGitGadget, Junio C Hamano, Todd Zullinger, Bagas Sanjaya.
Thread: https://gitlist.dev/t/58857

## Jean-Noël Avila via GitGitGadget, 2022-11-26 17:24

Subject: [PATCH] i18n: fix command template placeholder format
Message-ID: <pull.1435.git.1669483442230.gitgitgadget@gmail.com>
URL: https://gitlist.dev/e/pull.1435.git.1669483442230.gitgitgadget%40gmail.com

```
From: =?UTF-8?q?Jean-No=C3=ABl=20Avila?= <jn.avila@free.fr>

Signed-off-by: Jean-Noël Avila <jn.avila@free.fr>
---
    i18n: fix command template placeholder format
    
    This patch applies to the revision 2.39 rc0 during the l10n process.

Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1435%2Fjnavila%2Ffix_i18n_v2.39-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1435/jnavila/fix_i18n_v2.39-v1
Pull-Request: https://github.com/gitgitgadget/git/pull/1435

 Documentation/git-revert.txt | 2 +-
 builtin/revert.c             | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/git-revert.txt b/Documentation/git-revert.txt
index 5016755efb6..d2e10d3dceb 100644
--- a/Documentation/git-revert.txt
+++ b/Documentation/git-revert.txt
@@ -8,7 +8,7 @@ git-revert - Revert some existing commits
 SYNOPSIS
 --------
 [verse]
-'git revert' [--[no-]edit] [-n] [-m parent-number] [-s] [-S[<keyid>]] <commit>...
+'git revert' [--[no-]edit] [-n] [-m <parent-number>] [-s] [-S[<keyid>]] <commit>...
 'git revert' (--continue | --skip | --abort | --quit)
 
 DESCRIPTION
diff --git a/builtin/revert.c b/builtin/revert.c
index ee32c714a76..6a9b550a61f 100644
--- a/builtin/revert.c
+++ b/builtin/revert.c
@@ -21,7 +21,7 @@
  */
 
 static const char * const revert_usage[] = {
-	N_("git revert [--[no-]edit] [-n] [-m parent-number] [-s] [-S[<keyid>]] <commit>..."),
+	N_("git revert [--[no-]edit] [-n] [-m <parent-number>] [-s] [-S[<keyid>]] <commit>..."),
 	N_("git revert (--continue | --skip | --abort | --quit)"),
 	NULL
 };

base-commit: c000d916380bb59db69c78546928eadd076b9c7d
-- 
gitgitgadget

```

## Junio C Hamano, 2022-11-27 01:27

Subject: Re: [PATCH] i18n: fix command template placeholder format
Message-ID: <xmqqr0xp9of5.fsf@gitster.g>
URL: https://gitlist.dev/e/xmqqr0xp9of5.fsf%40gitster.g
In-Reply-To: <pull.1435.git.1669483442230.gitgitgadget@gmail.com>

```
"Jean-Noël Avila via GitGitGadget"  <gitgitgadget@gmail.com> writes:

>     i18n: fix command template placeholder format
>     
>     This patch applies to the revision 2.39 rc0 during the l10n process.

Thanks, will queue.

```

## Todd Zullinger, 2022-11-27 04:30

Subject: Re: [PATCH] i18n: fix command template placeholder format
Message-ID: <Y4Ln43f6iP1b6EHo@pobox.com>
URL: https://gitlist.dev/e/Y4Ln43f6iP1b6EHo%40pobox.com
In-Reply-To: <xmqqr0xp9of5.fsf@gitster.g>

```
Junio C Hamano wrote:
> "Jean-Noël Avila via GitGitGadget"  <gitgitgadget@gmail.com> writes:
> 
>>     i18n: fix command template placeholder format
>>     
>>     This patch applies to the revision 2.39 rc0 during the l10n process.
> 
> Thanks, will queue.

In case its helpful, an identical patch was sent on
Wednesday, Subject: [PATCH] revert: add angle bracket around
argument 'parent-number' in usage.

    <pull.1386.git.git.1669241489179.gitgitgadget@gmail.com>

Its commit message notes:

    8c9e292dc0 (doc txt & -h consistency: add missing
    options and labels, 2022-10-13) adds detailed usage line
    for revert and cherry-pick, which both contain a flag
    `-m` for parent number.

    Angle brackets shall be marked around parent-number,
    since it represents
    an argument.

-- 
Todd

```

## Bagas Sanjaya, 2022-11-28 08:37

Subject: Re: [PATCH] i18n: fix command template placeholder format
Message-ID: <abd9d2ce-a68b-905e-4ec2-4964aef6ee6c@gmail.com>
URL: https://gitlist.dev/e/abd9d2ce-a68b-905e-4ec2-4964aef6ee6c%40gmail.com
In-Reply-To: <pull.1435.git.1669483442230.gitgitgadget@gmail.com>

```
On 11/27/22 00:24, Jean-Noël Avila via GitGitGadget wrote:
 >  static const char * const revert_usage[] = {
> -	N_("git revert [--[no-]edit] [-n] [-m parent-number] [-s] [-S[<keyid>]] <commit>..."),
> +	N_("git revert [--[no-]edit] [-n] [-m <parent-number>] [-s] [-S[<keyid>]] <commit>..."),
>  	N_("git revert (--continue | --skip | --abort | --quit)"),
>  	NULL
>  };
> 

Looks OK, thanks!

Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com>

-- 
An old man doll... just what I always wanted! - Clara


```

## Junio C Hamano, 2022-11-28 13:41

Subject: Re: [PATCH] i18n: fix command template placeholder format
Message-ID: <xmqqpmd75h8v.fsf@gitster.g>
URL: https://gitlist.dev/e/xmqqpmd75h8v.fsf%40gitster.g
In-Reply-To: <Y4Ln43f6iP1b6EHo@pobox.com>

```
Todd Zullinger <tmz@pobox.com> writes:

>> Thanks, will queue.
>
> In case its helpful, an identical patch was sent on
> Wednesday, Subject: [PATCH] revert: add angle bracket around
> argument 'parent-number' in usage.

Yup, more eyes finding the same issue is very good and is very much
appreciated.  And in a case like this trivial one, it is not
surprising that there is only a single way to make it correct.

Thanks, all, again for your contributions.


```