Ævar Arnfjörð Bjarmason wrote: > --- a/builtin/init-db.c > +++ b/builtin/init-db.c > @@ -31,7 +31,7 @@ static void safe_create_dir(const char *dir, int share) > } > } > else if (share && adjust_shared_perm(dir)) > - die("Could not make %s writable by group", dir); > + die(_("Could not make %s writable by group"), dir); Sensible. I wonder if die() should not just be taught to automatically look up translations for its format string (could gettext handle that?). Although we try not to change plumbing error messages without good reason, details of error messages change often enough that imvho scripts should not be relying on them. > - die_errno("cannot stat '%s'", path); > + die_errno(_("cannot stat '%s'"), path); Will strerror() cope correctly without LC_CTYPE set up? (Not part of this series, just something I was reminded of.) -- 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