summaryrefslogtreecommitdiff
path: root/sequencer.c
diff options
context:
space:
mode:
authorElijah Newren <newren@gmail.com>2020-03-11 15:30:22 (GMT)
committerJunio C Hamano <gitster@pobox.com>2020-03-11 19:11:05 (GMT)
commit9a1b7474d6babad5fabed9a2926a57d875f3820b (patch)
tree865d615e5582efc65b870e07eb91f3ead0d573b2 /sequencer.c
parent937d1436302582c4e769594a805978d03e89ad59 (diff)
downloadgit-9a1b7474d6babad5fabed9a2926a57d875f3820b.zip
git-9a1b7474d6babad5fabed9a2926a57d875f3820b.tar.gz
git-9a1b7474d6babad5fabed9a2926a57d875f3820b.tar.bz2
sequencer: clear state upon dropping a become-empty commit
In commit e98c4269c8 ("rebase (interactive-backend): fix handling of commits that become empty", 2020-02-15), the merge backend was changed to drop commits that did not start empty but became so after being applied (because their changes were a subset of what was already upstream). This new code path did not need to go through the process of creating a commit, since we were dropping the commit instead. Unfortunately, this also means we bypassed the clearing of the CHERRY_PICK_HEAD and MERGE_MSG files, which if there were no further commits to cherry-pick would mean that the rebase would end but assume there was still an operation in progress. Ensure that we clear such state files when we decide to drop the commit. Signed-off-by: Elijah Newren <newren@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'sequencer.c')
-rw-r--r--sequencer.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/sequencer.c b/sequencer.c
index f8e242b..df89ff4 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -1955,6 +1955,8 @@ static int do_pick_commit(struct repository *r,
flags |= ALLOW_EMPTY;
} else if (allow == 2) {
drop_commit = 1;
+ unlink(git_path_cherry_pick_head(r));
+ unlink(git_path_merge_msg(r));
fprintf(stderr,
_("dropping %s %s -- patch contents already upstream\n"),
oid_to_hex(&commit->object.oid), msg.subject);