Re: [PATCH v4 07/11] builtin/diagnose.c: create 'git diagnose' builtin
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Aug 18, 2022, 19:12 UTC
- Message-ID
- <xmqqzgg1nz6v.fsf@gitster.g>
- In-Reply-To
- <220818.86edxd8k8q.gmgdl@evledraar.gmail.com>
Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
Show 15 quoted lines
> On Fri, Aug 12 2022, Victoria Dye via GitGitGadget wrote:
>
>> From: Victoria Dye <vdye@github.com>
>> [...]
>
> This is correct:
>
>> +'git diagnose' [(-o | --output-directory) <path>] [(-s | --suffix) <format>]
>
> ...
>
>> + N_("git diagnose [-o|--output-directory <path>] [-s|--suffix <format>]"),
>
> But this is not, it's missing () around the short v.s. long option, and
> we should have a space surrounding the "|" as well.You are commenting on what appears inside N_() and I agree that it should match the other one.
It is kind of sad that our usage strings do not allow proper translation and instead force translaters to _know_ that they are supposed to touch _only_ the placeholder strings (path and format in this case).
Not your fault, or Victoria's, of course ;-)