Skip to content

Commit

Permalink
rebase: reorder validation steps
Browse files Browse the repository at this point in the history
Reorder validation steps in preparation for the validation to be factored
out from git-rebase--interactive.sh into git-rebase.sh.

The main functional difference is that the pre-rebase hook will no longer
be run if the work tree is dirty.

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 Junio C Hamano committed Feb 10, 2011
1 parent f3889b8 commit 8f9bfb6
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
4 changes: 2 additions & 2 deletions git-rebase--interactive.sh
Original file line number Diff line number Diff line change
Expand Up @@ -792,12 +792,12 @@ else
test -z "$onto" &&
die "You must specify --onto when using --root"
fi
require_clean_work_tree "rebase" "Please commit or stash them."

run_pre_rebase_hook "$upstream_arg" "$@"

comment_for_reflog start

require_clean_work_tree "rebase" "Please commit or stash them."

if test ! -z "$1"
then
output git checkout "$1" -- ||
Expand Down
10 changes: 5 additions & 5 deletions git-rebase.sh
Original file line number Diff line number Diff line change
Expand Up @@ -433,8 +433,6 @@ fi

test "$type" = interactive && run_interactive_rebase "$@"

require_clean_work_tree "rebase" "Please commit or stash them."

if test -z "$rebase_root"
then
# The upstream head must be given. Make sure it is valid.
Expand Down Expand Up @@ -476,9 +474,6 @@ case "$onto_name" in
;;
esac

# If a hook exists, give it a chance to interrupt
run_pre_rebase_hook "$upstream_arg" "$@"

# If the branch to rebase is given, that is the branch we will rebase
# $branch_name -- branch being rebased, or HEAD (already detached)
# $orig_head -- commit object name of tip of the branch before rebasing
Expand Down Expand Up @@ -517,6 +512,8 @@ case "$#" in
esac
orig_head=$branch

require_clean_work_tree "rebase" "Please commit or stash them."

# Now we are rebasing commits $upstream..$branch (or with --root,
# everything leading up to $branch) on top of $onto

Expand All @@ -538,6 +535,9 @@ then
fi
fi

# If a hook exists, give it a chance to interrupt
run_pre_rebase_hook "$upstream_arg" "$@"

# Detach HEAD and reset the tree
say "First, rewinding head to replay your work on top of it..."
git checkout -q "$onto^0" || die "could not detach HEAD"
Expand Down

0 comments on commit 8f9bfb6

Please sign in to comment.