]> git.ipfire.org Git - thirdparty/git.git/commit - merge-recursive.c
submodules: allow empty working-tree dirs in merge/cherry-pick
authorDavid Turner <dturner@twosigma.com>
Mon, 7 Nov 2016 18:31:31 +0000 (13:31 -0500)
committerJunio C Hamano <gitster@pobox.com>
Fri, 18 Nov 2016 04:25:54 +0000 (20:25 -0800)
commit5423d2e7005eca89481d3137569b2b96b4d133ff
tree40cf180af1c7e90518471610df7e269a2adaef18
parent3ab228137f980ff72dbdf5064a877d07bec76df9
submodules: allow empty working-tree dirs in merge/cherry-pick

When a submodule is being merged or cherry-picked into a working
tree that already contains a corresponding empty directory, do not
record a conflict.

One situation where this bug appears is:

- Commit 1 adds a submodule
- Commit 2 removes that submodule and re-adds it into a subdirectory
       (sub1 to sub1/sub1).
- Commit 3 adds an unrelated file.

Now the user checks out commit 1 (first deinitializing the submodule),
and attempts to cherry-pick commit 3.  Previously, this would fail,
because the incoming submodule sub1/sub1 would falsely conflict with
the empty sub1 directory.

This patch ignores the empty sub1 directory, fixing the bug.  We only
ignore the empty directory if the object being emplaced is a
submodule, which expects an empty directory.

Signed-off-by: David Turner <dturner@twosigma.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
merge-recursive.c
t/t3030-merge-recursive.sh
t/t3426-rebase-submodule.sh