Re: [PATCH 20/20] gettextize: git-am printf(1) message to eval_gettext

[Jonathan Nieder <jrnieder@xxxxxxxxx>]

Hi,

Ævar Arnfjörð Bjarmason wrote:

> +++ b/git-am.sh
> @@ -778,7 +778,7 @@ did you forget to use 'git add'?"; echo
>        fi
>        if test $apply_status != 0
>        then
> -               printf 'Patch failed at %s %s\n' "$msgnum" "$FIRSTLINE"
> +               echo "$(eval_gettext "Patch failed at \$msgnum \$FIRSTLINE")"

Probably I am missing something silly, but why not just:

        eval_gettext 'Patch failed at $msgnum $FIRSTLINE\n'

? I ask because it is nice to be able to avoid "echo" with arbitrary
data: special characters (e.g. \) can make for portability hassles.

Jonathan
--
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