[RFC PATCH 2/2] submodule update: continue when a checkout fails
From: Fredrik Gustafsson <hidden>
Date: 2016-06-15 22:51:14
Subsystem:
the rest · Maintainer:
Linus Torvalds
When running git submodule update the submodules are checked out in alphabetic order. When an update of a submodule fails because of a checkout error, continue to the next submodule and when done with all submodules, exit with an error. We only do this for 'safe' case when a submodule is not marked as rebase or merge. When the update of a submodule fails because of a merge or rebase, the update will still die immediately to give the user an opportunity to resolve any conflicts before continuing. Since submodule 'b' does not necessarily need to be dependent on submodule 'a' this behavior is helpful if we have a lot of submodules. For example if some submodules currently experience network problems we can securely continue with the other submodules and the user can revisit the failed one later on. It also is helpful if a checkout fails because a submodules working directory is dirty. Now the user can cleanup the submodule in question and another git submodule update will just update the failed submodule instead of all submodules that are ordered alphabetically afterwards. Signed-off-by: Fredrik Gustafsson <redacted> Mentored-by: Jens Lehmann [off-list ref] Mentored-by: Heiko Voigt [off-list ref] --- git-submodule.sh | 42 ++++++++++++++++++++++++++++++++++++------ t/t7406-submodule-update.sh | 29 +++++++++++++++++++++++++++++ 2 files changed, 65 insertions(+), 6 deletions(-)
diff --git a/git-submodule.sh b/git-submodule.sh
index bf110e9..02c41c7 100755
--- a/git-submodule.sh
+++ b/git-submodule.sh@@ -444,7 +444,8 @@ cmd_update() fi cloned_modules= - module_list "$@" | + module_list "$@" | { + err= while read mode sha1 stage path do if test "$stage" = U
@@ -525,17 +526,46 @@ cmd_update() ;; esac - (clear_local_git_env; cd "$path" && $command "$sha1") || - die "Unable to $action '$sha1' in submodule path '$path'" - say "Submodule path '$path': $msg '$sha1'" + if (clear_local_git_env; cd "$path" && $command "$sha1") + then + say "Submodule path '$path': $msg '$sha1'" + else + case $action in + rebase|merge) + die "Unable to $action '$sha1' in submodule path '$path'" 2 + ;; + *) + say "Unable to $action '$sha1' in submodule path '$path'" + err="Failed to $action one or more submodule(s)" + continue + ;; + esac + fi fi if test -n "$recursive" then - (clear_local_git_env; cd "$path" && eval cmd_update "$orig_flags") || - die "Failed to recurse into submodule path '$path'" + (clear_local_git_env; cd "$path" && eval cmd_update "$orig_flags") + res=$? + if test $res -gt 0 + then + if test $res -eq 1 + then + say "Failed to recurse into submodule path '$path'" + continue + else + die "Failed to recurse into submodule path '$path'" $res + fi + fi fi done + + if test -n "$err" + then + die "$err" + fi + + } } set_name_rev () {
diff --git a/t/t7406-submodule-update.sh b/t/t7406-submodule-update.sh
index 4f16fcc..e79c4df 100755
--- a/t/t7406-submodule-update.sh
+++ b/t/t7406-submodule-update.sh@@ -298,4 +298,33 @@ test_expect_success 'submodule update ignores update=rebase config for new submo ) ' +test_expect_success 'submodule update continues after checkout error' ' + (cd super && + git reset --hard HEAD && + git submodule add ../submodule submodule2 && + git submodule init && + git commit -am "new_submodule" && + (cd submodule2 && + git rev-parse HEAD > ../expect + ) && + (cd submodule && + test_commit "update_submodule" file + ) && + (cd submodule2 && + test_commit "update_submodule2" file + ) && + git add submodule && + git add submodule2 && + git commit -m "two_new_submodule_commits" && + (cd submodule && + echo "" > file + ) && + git checkout HEAD^ && + test_must_fail git submodule update && + (cd submodule2 && + git rev-parse HEAD > ../actual + ) && + test_cmp expect actual + ) +' test_done
--
1.7.5.1.219.ge2152.dirty