Code

revert --abort: do not leave behind useless sequencer-old directory
authorJonathan Nieder <jrnieder@gmail.com>
Wed, 23 Nov 2011 10:04:52 +0000 (04:04 -0600)
committerJunio C Hamano <gitster@pobox.com>
Wed, 23 Nov 2011 18:56:29 +0000 (10:56 -0800)
commitb15aa973b296ca36ae39592491bcb02944ac0f7a
treec91043adc99f82b00d4a1eed37eea0aa5ea5f0b5
parent2a4037d0a745134a3b94cb3d60d2793a26bd9027
revert --abort: do not leave behind useless sequencer-old directory

The "git cherry-pick --abort" command currently renames the
.git/sequencer directory to .git/sequencer-old instead of removing it
on success due to an accident.  cherry-pick --abort is designed to
work in three steps:

 1) find which commit to roll back to
 2) call "git reset --merge <commit>" to move to that commit
 3) remove the .git/sequencer directory

But the careless author forgot step 3 entirely.  The only reason the
command worked anyway is that "git reset --merge <commit>" renames the
.git/sequencer directory as a secondary effect --- after moving to
<commit>, or so the logic goes, it is unlikely but possible that the
caller of git reset wants to continue the series of cherry-picks that
was in progress, so git renames the sequencer state to
.git/sequencer-old to be helpful while allowing the cherry-pick to be
resumed if the caller did not want to end the sequence after all.

By running "git cherry-pick --abort", the operator has clearly
indicated that she is not planning to continue cherry-picking.  Remove
the (renamed) .git/sequencer directory as intended all along.

Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/revert.c
t/t7106-reset-sequence.sh