From: Stefano Lattarini Date: Wed, 25 Jul 2012 09:59:20 GMT Subject: Re: [PATCH v4 5/7] i18n: am: mark more strings for translation Message-ID: <500FC378.5090706@gmail.com> In-Reply-To: <1493bd09efd0ba6e1d56500305d989047da245ec.1343188013.git.worldhello.net@gmail.com> Sorry to be so nit-picky, but ... On 07/25/2012 05:53 AM, Jiang Xin wrote: > Mark strings in 'git-am.sh' for translation. In the last chunk, I > changed '$1' to '-b/--binary' for this reason: > > * First, if there is a variable in the l10n message, we could not use > gettext. Because the variable will be expanded (evaluated) and will > never match the entry in the po file. > > * Second, if there is a positional parameter ($1, $2,...) in the > message, we could not use eval_gettext either. Because > eval_gettext may be a wapper for gettext, and the positional > parameter would loose it's original context. > ... here you should s/it's/its/. Regards, Stefano