From: Nguyen Thai Ngoc Duy Date: Wed, 30 May 2012 05:27:55 GMT Subject: Re: [PATCH 03/65] i18n: archive: mark parseopt strings for translation Message-ID: In-Reply-To: <20120529203759.GC22151@burratino> On Wed, May 30, 2012 at 3:37 AM, Jonathan Nieder wrote: > Junio C Hamano wrote: >> Nguyễn Thái Ngọc Duy   writes: > >>> --- a/archive.c >>> +++ b/archive.c >>> @@ -7,10 +7,10 @@ >>>  #include "unpack-trees.h" >>> >>>  static char const * const archive_usage[] = { >>> -    "git archive [options] [...]", >>> -    "git archive --list", >>> -    "git archive --remote [--exec ] [options] [...]", >>> -    "git archive --remote [--exec ] --list", >>> +    N_("git archive [options] [...]"), >>> +    N_("git archive --list"), >>> +    N_("git archive --remote [--exec ] [options] [...]"), >>> +    N_("git archive --remote [--exec ] --list"), >>>      NULL >>>  }; >> >> I can understand the "help" text member of OPT_BLAH() construct, but >> do we really want to translate the above? > > , , [options], , and may be spelled > differently in other languages.  Would it be possible to take care > of this kind of substitution automatically? not for the "[...]" part > At least "git archive --list" should probably be kept untranslated > to lessen the translator workload a bit.  Good catch. I use po-mode and for strings that no change is required like this, simply C-j to copy as-is after a glance. -- Duy