On Fri, Jan 18, 2019 at 3:18 PM Junio C Hamano <gitster@xxxxxxxxx> wrote: > > Stefan Beller <sbeller@xxxxxxxxxx> writes: > > > 74d4731da1f (submodule--helper: replace connect-gitdir-workingtree by > > ensure-core-worktree, 2018-08-13) forgot to exit the submodule operation > > if the helper could not ensure that core.worktree is set correctly. > > > > Signed-off-by: Stefan Beller <sbeller@xxxxxxxxxx> > > --- > > git-submodule.sh | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/git-submodule.sh b/git-submodule.sh > > index 5e608f8bad3..02a0d728174 100755 > > --- a/git-submodule.sh > > +++ b/git-submodule.sh > > @@ -548,7 +548,7 @@ cmd_update() > > do > > die_if_unmatched "$quickabort" "$sha1" > > > > - git submodule--helper ensure-core-worktree "$sm_path" > > + git submodule--helper ensure-core-worktree "$sm_path" || exit 1 > > Good to have more places check errors from underlying command. > > Thanks. > > Out of curiousity, was this found by code inspection, or was there a > real-world breakage episode? By code inspection to come up with an answer for https://public-inbox.org/git/CAG0vfyQeA3Hm7AsYgYtP4v-Yg0=rKXW0YYfg_emAwEscZha4VA@xxxxxxxxxxxxxx/ That issue in itself is not fully solved (Duy is still looking into it), but this fixes a minor inconsistency along the way.
