merge-recursive: never leave index unmerged while recursing
When you are trying to come up with the final result (i.e. depth=0), you
want to record how the conflict arose by registering the state of the
common ancestor, your branch and the other branch in the index, hence you
want to do update_stages().
When you are merging with positive depth, that is because of a criss-cross
merge situation. In such a case, you would need to record the tentative
result, with conflict markers and all, as if the merge went cleanly, even
if there are conflicts, in order to write it out as a tree object later to
be used as a common ancestor tree.
update_file() calls update_file_flags() with update_cache=1 to signal that
the result needs to be written to the index at stage #0 (i.e. merged), and
the code should not clobber the index further by calling update_stages().
The codepath to deal with rename/delete conflict in a recursive merge
however left the index unmerged.
Signed-off-by: Dave Olszewski <cxreg@pobox.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2009-05-10 05:49:59 +08:00
|
|
|
#!/bin/sh
|
|
|
|
|
|
|
|
test_description='merge-recursive backend test'
|
|
|
|
|
leak tests: mark passing SANITIZE=leak tests as leak-free
Mark those remaining tests that pass when run under SANITIZE=leak with
TEST_PASSES_SANITIZE_LEAK=true, these were either omitted in
f346fcb62a0 (Merge branch 'ab/mark-leak-free-tests-even-more',
2021-12-15) and 5a4f8381b68 (Merge branch 'ab/mark-leak-free-tests',
2021-10-25), or have had their memory leaks fixed since then.
With this change there's now a a one-to-one mapping between those
tests that we have opted-in via "TEST_PASSES_SANITIZE_LEAK=true", and
those that pass with the new "check" mode:
GIT_TEST_PASSING_SANITIZE_LEAK=check \
GIT_TEST_SANITIZE_LEAK_LOG=true \
make test SANITIZE=leak
Note that the "GIT_TEST_SANITIZE_LEAK_LOG=true" is needed due to the
edge cases noted in a preceding commit, i.e. in some cases we'd pass
the test itself, but still have outstanding leaks due to ignored exit
codes.
The "GIT_TEST_SANITIZE_LEAK_LOG=true" corrects for that, we're only
marking those tests as passing that really don't have any leaks,
whether that was reflected in their exit code or not.
Note that the change here to "t9100-git-svn-basic.sh" is marking that
test as passing under SANITIZE=leak, we're removing a
"TEST_FAILS_SANITIZE_LEAK=true" line, not
"TEST_PASSES_SANITIZE_LEAK=true". See 7a98d9ab00d (revisions API: have
release_revisions() release "cmdline", 2022-04-13) for the
introduction of that t/lib-git-svn.sh-specific variable.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2022-07-28 07:13:41 +08:00
|
|
|
TEST_PASSES_SANITIZE_LEAK=true
|
merge-recursive: never leave index unmerged while recursing
When you are trying to come up with the final result (i.e. depth=0), you
want to record how the conflict arose by registering the state of the
common ancestor, your branch and the other branch in the index, hence you
want to do update_stages().
When you are merging with positive depth, that is because of a criss-cross
merge situation. In such a case, you would need to record the tentative
result, with conflict markers and all, as if the merge went cleanly, even
if there are conflicts, in order to write it out as a tree object later to
be used as a common ancestor tree.
update_file() calls update_file_flags() with update_cache=1 to signal that
the result needs to be written to the index at stage #0 (i.e. merged), and
the code should not clobber the index further by calling update_stages().
The codepath to deal with rename/delete conflict in a recursive merge
however left the index unmerged.
Signed-off-by: Dave Olszewski <cxreg@pobox.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2009-05-10 05:49:59 +08:00
|
|
|
. ./test-lib.sh
|
|
|
|
|
|
|
|
# A <- create some files
|
|
|
|
# / \
|
|
|
|
# B C <- cause rename/delete conflicts between B and C
|
|
|
|
# / \
|
|
|
|
# |\ /|
|
|
|
|
# | D E |
|
|
|
|
# | \ / |
|
|
|
|
# | X |
|
|
|
|
# | / \ |
|
|
|
|
# | / \ |
|
|
|
|
# |/ \|
|
|
|
|
# F G <- merge E into B, D into C
|
|
|
|
# \ /
|
|
|
|
# \ /
|
|
|
|
# \ /
|
|
|
|
# H <- recursive merge crashes
|
|
|
|
#
|
|
|
|
|
|
|
|
# initialize
|
|
|
|
test_expect_success 'setup repo with criss-cross history' '
|
|
|
|
mkdir data &&
|
|
|
|
|
|
|
|
# create a bunch of files
|
|
|
|
n=1 &&
|
|
|
|
while test $n -le 10
|
|
|
|
do
|
|
|
|
echo $n > data/$n &&
|
|
|
|
n=$(($n+1)) ||
|
2015-03-25 13:29:52 +08:00
|
|
|
return 1
|
merge-recursive: never leave index unmerged while recursing
When you are trying to come up with the final result (i.e. depth=0), you
want to record how the conflict arose by registering the state of the
common ancestor, your branch and the other branch in the index, hence you
want to do update_stages().
When you are merging with positive depth, that is because of a criss-cross
merge situation. In such a case, you would need to record the tentative
result, with conflict markers and all, as if the merge went cleanly, even
if there are conflicts, in order to write it out as a tree object later to
be used as a common ancestor tree.
update_file() calls update_file_flags() with update_cache=1 to signal that
the result needs to be written to the index at stage #0 (i.e. merged), and
the code should not clobber the index further by calling update_stages().
The codepath to deal with rename/delete conflict in a recursive merge
however left the index unmerged.
Signed-off-by: Dave Olszewski <cxreg@pobox.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2009-05-10 05:49:59 +08:00
|
|
|
done &&
|
|
|
|
|
|
|
|
# check them in
|
|
|
|
git add data &&
|
|
|
|
git commit -m A &&
|
|
|
|
git branch A &&
|
|
|
|
|
|
|
|
# a file in one branch
|
|
|
|
git checkout -b B A &&
|
|
|
|
git rm data/9 &&
|
|
|
|
git add data &&
|
|
|
|
git commit -m B &&
|
|
|
|
|
|
|
|
# with a branch off of it
|
|
|
|
git branch D &&
|
|
|
|
|
|
|
|
# put some commits on D
|
|
|
|
git checkout D &&
|
|
|
|
echo testD > data/testD &&
|
|
|
|
git add data &&
|
|
|
|
git commit -m D &&
|
|
|
|
|
|
|
|
# back up to the top, create another branch and cause
|
|
|
|
# a rename conflict with the file we deleted earlier
|
|
|
|
git checkout -b C A &&
|
|
|
|
git mv data/9 data/new-9 &&
|
|
|
|
git add data &&
|
|
|
|
git commit -m C &&
|
|
|
|
|
|
|
|
# with a branch off of it
|
|
|
|
git branch E &&
|
|
|
|
|
|
|
|
# put a commit on E
|
|
|
|
git checkout E &&
|
|
|
|
echo testE > data/testE &&
|
|
|
|
git add data &&
|
|
|
|
git commit -m E &&
|
|
|
|
|
|
|
|
# now, merge E into B
|
|
|
|
git checkout B &&
|
|
|
|
test_must_fail git merge E &&
|
|
|
|
# force-resolve
|
|
|
|
git add data &&
|
|
|
|
git commit -m F &&
|
|
|
|
git branch F &&
|
|
|
|
|
|
|
|
# and merge D into C
|
|
|
|
git checkout C &&
|
|
|
|
test_must_fail git merge D &&
|
|
|
|
# force-resolve
|
|
|
|
git add data &&
|
|
|
|
git commit -m G &&
|
|
|
|
git branch G
|
|
|
|
'
|
|
|
|
|
2018-08-04 07:09:23 +08:00
|
|
|
test_expect_success 'recursive merge between F and G does not cause segfault' '
|
merge-recursive: never leave index unmerged while recursing
When you are trying to come up with the final result (i.e. depth=0), you
want to record how the conflict arose by registering the state of the
common ancestor, your branch and the other branch in the index, hence you
want to do update_stages().
When you are merging with positive depth, that is because of a criss-cross
merge situation. In such a case, you would need to record the tentative
result, with conflict markers and all, as if the merge went cleanly, even
if there are conflicts, in order to write it out as a tree object later to
be used as a common ancestor tree.
update_file() calls update_file_flags() with update_cache=1 to signal that
the result needs to be written to the index at stage #0 (i.e. merged), and
the code should not clobber the index further by calling update_stages().
The codepath to deal with rename/delete conflict in a recursive merge
however left the index unmerged.
Signed-off-by: Dave Olszewski <cxreg@pobox.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2009-05-10 05:49:59 +08:00
|
|
|
git merge F
|
|
|
|
'
|
|
|
|
|
|
|
|
test_done
|