Skip to content

Commit

Permalink
rebase: factor out sub command handling
Browse files Browse the repository at this point in the history
Factor out the common parts of the handling of the sub commands
'--continue', '--skip' and '--abort'. The '--abort' handling can
handled completely in git-rebase.sh.

After this refactoring, the calls to git-rebase--am.sh,
git-rebase--merge.sh and git-rebase--interactive.sh will be better
aligned. There will only be one call to interactive rebase that will
shortcut the very last part of git-rebase.sh.

Signed-off-by: Martin von Zweigbergk <martin.von.zweigbergk@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Martin von Zweigbergk authored and gitster committed Feb 10, 2011
1 parent 4974c2c commit 2959c28
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 33 deletions.
34 changes: 3 additions & 31 deletions git-rebase--interactive.sh
Original file line number Diff line number Diff line change
Expand Up @@ -509,9 +509,7 @@ do_next () {
test -s "$todo" && return

comment_for_reflog finish &&
head_name=$(cat "$state_dir"/head-name) &&
orig_head=$(cat "$state_dir"/head) &&
shortonto=$(git rev-parse --short $(cat "$state_dir"/onto)) &&
shortonto=$(git rev-parse --short $onto) &&
newhead=$(git rev-parse HEAD) &&
case $head_name in
refs/*)
Expand All @@ -521,7 +519,7 @@ do_next () {
;;
esac && {
test ! -f "$state_dir"/verbose ||
git diff-tree --stat $(cat "$state_dir"/head)..HEAD
git diff-tree --stat $orig_head..HEAD
} &&
{
test -s "$rewritten_list" &&
Expand Down Expand Up @@ -655,14 +653,6 @@ rearrange_squash () {
case "$action" in
continue)
get_saved_options
comment_for_reflog continue

# Sanity check
git rev-parse --verify HEAD >/dev/null ||
die "Cannot read HEAD"
git update-index --ignore-submodules --refresh &&
git diff-files --quiet --ignore-submodules ||
die "Working tree is dirty"

# do we have anything to commit?
if git diff-index --cached --quiet --ignore-submodules HEAD --
Expand Down Expand Up @@ -693,30 +683,12 @@ first and then run 'git rebase --continue' again."
require_clean_work_tree "rebase"
do_rest
;;
abort)
get_saved_options
comment_for_reflog abort

git rerere clear

head_name=$(cat "$state_dir"/head-name)
orig_head=$(cat "$state_dir"/head)
case $head_name in
refs/*)
git symbolic-ref HEAD $head_name
;;
esac &&
output git reset --hard $orig_head &&
rm -rf "$state_dir"
exit
;;
skip)
get_saved_options
comment_for_reflog skip

git rerere clear

output git reset --hard && do_rest
do_rest
;;
esac

Expand Down
17 changes: 15 additions & 2 deletions git-rebase.sh
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,12 @@ test "$(git config --bool rebase.autosquash)" = "true" && autosquash=t
read_basic_state () {
head_name=$(cat "$state_dir"/head-name) &&
onto=$(cat "$state_dir"/onto) &&
orig_head=$(cat "$state_dir"/orig-head) &&
if test "$type" = interactive
then
orig_head=$(cat "$state_dir"/head)
else
orig_head=$(cat "$state_dir"/orig-head)
fi &&
GIT_QUIET=$(cat "$state_dir"/quiet)
}

Expand Down Expand Up @@ -262,11 +267,19 @@ test $# -gt 2 && usage
if test -n "$action"
then
test -z "$in_progress" && die "No rebase in progress?"
test "$type" = interactive && run_specific_rebase
# Only interactive rebase uses detailed reflog messages
if test "$type" = interactive && test "$GIT_REFLOG_ACTION" = rebase
then
GIT_REFLOG_ACTION="rebase -i ($action)"
export GIT_REFLOG_ACTION
fi
fi

case "$action" in
continue)
# Sanity check
git rev-parse --verify HEAD >/dev/null ||
die "Cannot read HEAD"
git update-index --ignore-submodules --refresh &&
git diff-files --quiet --ignore-submodules || {
echo "You must edit all merge conflicts and then"
Expand Down

0 comments on commit 2959c28

Please sign in to comment.