Jiang Xin <worldhello....@gmail.com> writes:

> Jean-Noël Avila <jn.av...@free.fr> 于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.


Reply via email to