Jiang Xin <worldhello.net@xxxxxxxxx> writes: > Jean-Noël Avila <jn.avila@xxxxxxx> 于2018年8月24日周五 上午5:02写道: >> diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c >> index 2bcc70fdfe..b56028ba9d 100644 >> --- a/builtin/submodule--helper.c >> +++ b/builtin/submodule--helper.c >> @@ -542,7 +542,7 @@ static void runcommand_in_submodule_cb(const struct cache_entry *list_item, >> argv_array_pushv(&cpr.args, info->argv); >> >> if (run_command(&cpr)) >> - die(_("run_command returned non-zero status while" >> + die(_("run_command returned non-zero status while " >> "recursing in the nested submodules of %s\n."), > > Any specific reason to put a period after newline? > And there is another mistake just above this one: > > "run_command returned non-zero status for %s\n." Embarrassing X-< These are not something the "fix" introduces (rather it is what it still misses), though.