diff options
author | Junio C Hamano <gitster@pobox.com> | 2019-04-09 20:14:26 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2019-04-09 20:14:26 +0300 |
commit | 32414ceb854de44a9b4b74c975ae410713a17c99 (patch) | |
tree | 32b3a4956c97d6132fd0013bcd68d73fc68eb12b | |
parent | c063a537bedd3b2363494171de2bbd58bf5c66a9 (diff) | |
parent | bd5e567dc75abae58cb364d1e55cbaf67edb8c98 (diff) |
Merge branch 'jt/submodule-fetch-errmsg'
Error message update.
* jt/submodule-fetch-errmsg:
submodule: explain first attempt failure clearly
-rwxr-xr-x | git-submodule.sh | 2 | ||||
-rw-r--r-- | submodule.c | 7 |
2 files changed, 8 insertions, 1 deletions
diff --git a/git-submodule.sh b/git-submodule.sh index 514ede2596..2c0fb6d723 100755 --- a/git-submodule.sh +++ b/git-submodule.sh @@ -594,7 +594,7 @@ cmd_update() # is not reachable from a ref. is_tip_reachable "$sm_path" "$sha1" || fetch_in_submodule "$sm_path" $depth || - say "$(eval_gettext "Unable to fetch in submodule path '\$displaypath'")" + say "$(eval_gettext "Unable to fetch in submodule path '\$displaypath'; trying to directly fetch \$sha1:")" # Now we tried the usual fetch, but $sha1 may # not be reachable from any of the refs diff --git a/submodule.c b/submodule.c index 21cf50ca15..b16c0ecc95 100644 --- a/submodule.c +++ b/submodule.c @@ -1548,6 +1548,13 @@ static int fetch_finish(int retvalue, struct strbuf *err, struct oid_array *commits; if (retvalue) + /* + * NEEDSWORK: This indicates that the overall fetch + * failed, even though there may be a subsequent fetch + * by commit hash that might work. It may be a good + * idea to not indicate failure in this case, and only + * indicate failure if the subsequent fetch fails. + */ spf->result = 1; if (!task || !task->sub) |