Re: [PATCH v2 09/14] i18n: notes: mark error messages for translation
- From
Jean-Noël Avila <jn.avila@free.fr>
- Date
- Sep 12, 2016, 12:23 UTC
- Message-ID
- <83f72924-9a9d-dc1c-109f-341f0cd7ac96@free.fr>
- In-Reply-To
- <1473679802-31381-9-git-send-email-vascomalmeida@sapo.pt>
Le 12/09/2016 à 13:29, Vasco Almeida a écrit :
Show 18 quoted lines
> Signed-off-by: Vasco Almeida <vascomalmeida@sapo.pt>
> ---
> builtin/notes.c | 18 +++++++++---------
> 1 file changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/builtin/notes.c b/builtin/notes.c
> index f848b89..abacae2 100644
> --- a/builtin/notes.c
> +++ b/builtin/notes.c
> @@ -340,7 +340,7 @@ static struct notes_tree *init_notes_check(const char *subcommand,
>
> ref = (flags & NOTES_INIT_WRITABLE) ? t->update_ref : t->ref;
> if (!starts_with(ref, "refs/notes/"))
> - die("Refusing to %s notes in %s (outside of refs/notes/)",
> + die(_("Refusing to %s notes in %s (outside of refs/notes/)"),
> subcommand, ref);
> return t;
> }Not sure this one will be easy to localize. The verb is passed as a parameter : see line 366 "list", line 426 "add", line 517 "copy", line 658 "show", line 816 "merge", line 908 "remove" or line 595 with argv[0].
If all the verbs are real subcommands, then the translators should be warned that this is some english twisting, but that they need to refer to the subcommand on the command line.
Otherwise,
Acked-by: Jean-Noël Avila <jean-noel.avila@free.fr>
JN