Re: [PATCH 10/51] i18n: git-am core say messages

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Apr 3, 2011 at 19:33, Jakub Narebski <jnareb@xxxxxxxxx> wrote:
> Ãvar ArnfjÃrà Bjarmason Â<avarab@xxxxxxxxx> writes:
>
>> Make the core git-am messages that use say() translatable. These are
>> visible on almost every git am invocation.
>>
>> There are tests that depend on the "Applying" output that need to be
>> skipped along with this translation.
>
>
>> - Â Â say "Applying: $FIRSTLINE"
>> + Â Â say "$(eval_gettext "Applying: \$FIRSTLINE")"
>
>> - Â Â Â Â Â Â Â Â Â Â say No changes -- Patch already applied.
>> + Â Â Â Â Â Â Â Â Â Â say "$(gettext "No changes -- Patch already applied.")"
>
> I guess that you canot simply make 'say' to do the translation because
> one doesn't know if getext or eval_gettext is needed, isn't it?

Indeed, the xgettext() rule couldn't tell the difference. And even if
we only used say() for gettext() I'd prefer not to make things more
complex by supporting gettext extraction from our own custom
functions. I'd like to be able to grep for gettext and get all the
invocations.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]