git-p4: better error reporting when p4 fails
[gitweb.git] / git-rebase--interactive.sh
index a79330f4522dcb6730de1fb55af9a55781406ba6..9947e6265fe7c0bc4531fd928bbf2373a0ae7552 100644 (file)
@@ -285,7 +285,7 @@ pick_one () {
                pick_one_preserving_merges "$@" && return
        output eval git cherry-pick $allow_rerere_autoupdate $allow_empty_message \
                        ${gpg_sign_opt:+$(git rev-parse --sq-quote "$gpg_sign_opt")} \
-                       "$strategy_args" $empty_args $ff "$@"
+                       $signoff "$strategy_args" $empty_args $ff "$@"
 
        # If cherry-pick dies it leaves the to-be-picked commit unrecorded. Reschedule
        # previous task so this commit is not lost.
@@ -524,10 +524,10 @@ do_pick () {
                # resolve before manually running git commit --amend then git
                # rebase --continue.
                git commit --allow-empty --allow-empty-message --amend \
-                          --no-post-rewrite -n -q -C $sha1 &&
+                          --no-post-rewrite -n -q -C $sha1 $signoff &&
                        pick_one -n $sha1 &&
                        git commit --allow-empty --allow-empty-message \
-                                  --amend --no-post-rewrite -n -q -C $sha1 \
+                                  --amend --no-post-rewrite -n -q -C $sha1 $signoff \
                                   ${gpg_sign_opt:+"$gpg_sign_opt"} ||
                                   die_with_patch $sha1 "$(eval_gettext "Could not apply \$sha1... \$rest")"
        else
@@ -740,8 +740,20 @@ get_missing_commit_check_level () {
        printf '%s' "$check_level" | tr 'A-Z' 'a-z'
 }
 
-git_rebase__interactive () {
-       case "$action" in
+# Initiate an action. If the cannot be any
+# further action it  may exec a command
+# or exit and not return.
+#
+# TODO: Consider a cleaner return model so it
+# never exits and always return 0 if process
+# is complete.
+#
+# Parameter 1 is the action to initiate.
+#
+# Returns 0 if the action was able to complete
+# and if 1 if further processing is required.
+initiate_action () {
+       case "$1" in
        continue)
                if test ! -d "$rewritten"
                then
@@ -836,8 +848,13 @@ To continue rebase after editing, run:
        show-current-patch)
                exec git show REBASE_HEAD --
                ;;
+       *)
+               return 1 # continue
+               ;;
        esac
+}
 
+setup_reflog_action () {
        comment_for_reflog start
 
        if test ! -z "$switch_to"
@@ -848,36 +865,18 @@ To continue rebase after editing, run:
 
                comment_for_reflog start
        fi
+}
 
+init_basic_state () {
        orig_head=$(git rev-parse --verify HEAD) || die "$(gettext "No HEAD?")"
        mkdir -p "$state_dir" || die "$(eval_gettext "Could not create temporary \$state_dir")"
        rm -f "$(git rev-parse --git-path REBASE_HEAD)"
 
        : > "$state_dir"/interactive || die "$(gettext "Could not mark as interactive")"
        write_basic_state
-       if test t = "$preserve_merges"
-       then
-               if test -z "$rebase_root"
-               then
-                       mkdir "$rewritten" &&
-                       for c in $(git merge-base --all $orig_head $upstream)
-                       do
-                               echo $onto > "$rewritten"/$c ||
-                                       die "$(gettext "Could not init rewritten commits")"
-                       done
-               else
-                       mkdir "$rewritten" &&
-                       echo $onto > "$rewritten"/root ||
-                               die "$(gettext "Could not init rewritten commits")"
-               fi
-               # No cherry-pick because our first pass is to determine
-               # parents to rewrite and skipping dropped commits would
-               # prematurely end our probe
-               merges_option=
-       else
-               merges_option="--no-merges --cherry-pick"
-       fi
+}
 
+init_revisions_and_shortrevisions () {
        shorthead=$(git rev-parse --short $orig_head)
        shortonto=$(git rev-parse --short $onto)
        if test -z "$rebase_root"
@@ -890,76 +889,9 @@ To continue rebase after editing, run:
                revisions=$onto...$orig_head
                shortrevisions=$shorthead
        fi
-       if test t != "$preserve_merges"
-       then
-               git rebase--helper --make-script ${keep_empty:+--keep-empty} \
-                       $revisions ${restrict_revision+^$restrict_revision} >"$todo" ||
-               die "$(gettext "Could not generate todo list")"
-       else
-               format=$(git config --get rebase.instructionFormat)
-               # the 'rev-list .. | sed' requires %m to parse; the instruction requires %H to parse
-               git rev-list $merges_option --format="%m%H ${format:-%s}" \
-                       --reverse --left-right --topo-order \
-                       $revisions ${restrict_revision+^$restrict_revision} | \
-                       sed -n "s/^>//p" |
-               while read -r sha1 rest
-               do
-
-                       if test -z "$keep_empty" && is_empty_commit $sha1 && ! is_merge_commit $sha1
-                       then
-                               comment_out="$comment_char "
-                       else
-                               comment_out=
-                       fi
-
-                       if test -z "$rebase_root"
-                       then
-                               preserve=t
-                               for p in $(git rev-list --parents -1 $sha1 | cut -d' ' -s -f2-)
-                               do
-                                       if test -f "$rewritten"/$p
-                                       then
-                                               preserve=f
-                                       fi
-                               done
-                       else
-                               preserve=f
-                       fi
-                       if test f = "$preserve"
-                       then
-                               touch "$rewritten"/$sha1
-                               printf '%s\n' "${comment_out}pick $sha1 $rest" >>"$todo"
-                       fi
-               done
-       fi
-
-       # Watch for commits that been dropped by --cherry-pick
-       if test t = "$preserve_merges"
-       then
-               mkdir "$dropped"
-               # Save all non-cherry-picked changes
-               git rev-list $revisions --left-right --cherry-pick | \
-                       sed -n "s/^>//p" > "$state_dir"/not-cherry-picks
-               # Now all commits and note which ones are missing in
-               # not-cherry-picks and hence being dropped
-               git rev-list $revisions |
-               while read rev
-               do
-                       if test -f "$rewritten"/$rev &&
-                          ! sane_grep "$rev" "$state_dir"/not-cherry-picks >/dev/null
-                       then
-                               # Use -f2 because if rev-list is telling us this commit is
-                               # not worthwhile, we don't want to track its multiple heads,
-                               # just the history of its first-parent for others that will
-                               # be rebasing on top of it
-                               git rev-list --parents -1 $rev | cut -d' ' -s -f2 > "$dropped"/$rev
-                               sha1=$(git rev-list -1 $rev)
-                               sane_grep -v "^[a-z][a-z]* $sha1" <"$todo" > "${todo}2" ; mv "${todo}2" "$todo"
-                               rm "$rewritten"/$rev
-                       fi
-               done
-       fi
+}
 
+complete_action() {
        test -s "$todo" || echo noop >> "$todo"
        test -z "$autosquash" || git rebase--helper --rearrange-squash || exit
        test -n "$cmd" && git rebase--helper --add-exec-commands "$cmd"
@@ -1018,3 +950,110 @@ EOF
        fi
        do_rest
 }
+
+git_rebase__interactive () {
+       initiate_action "$action"
+       ret=$?
+       if test $ret = 0; then
+               return 0
+       fi
+
+       setup_reflog_action
+       init_basic_state
+
+       init_revisions_and_shortrevisions
+
+       git rebase--helper --make-script ${keep_empty:+--keep-empty} \
+               $revisions ${restrict_revision+^$restrict_revision} >"$todo" ||
+       die "$(gettext "Could not generate todo list")"
+
+       complete_action
+}
+
+git_rebase__interactive__preserve_merges () {
+       initiate_action "$action"
+       ret=$?
+       if test $ret = 0; then
+               return 0
+       fi
+
+       setup_reflog_action
+       init_basic_state
+
+       if test -z "$rebase_root"
+       then
+               mkdir "$rewritten" &&
+               for c in $(git merge-base --all $orig_head $upstream)
+               do
+                       echo $onto > "$rewritten"/$c ||
+                               die "$(gettext "Could not init rewritten commits")"
+               done
+       else
+               mkdir "$rewritten" &&
+               echo $onto > "$rewritten"/root ||
+                       die "$(gettext "Could not init rewritten commits")"
+       fi
+
+       init_revisions_and_shortrevisions
+
+       format=$(git config --get rebase.instructionFormat)
+       # the 'rev-list .. | sed' requires %m to parse; the instruction requires %H to parse
+       git rev-list --format="%m%H ${format:-%s}" \
+               --reverse --left-right --topo-order \
+               $revisions ${restrict_revision+^$restrict_revision} | \
+               sed -n "s/^>//p" |
+       while read -r sha1 rest
+       do
+               if test -z "$keep_empty" && is_empty_commit $sha1 && ! is_merge_commit $sha1
+               then
+                       comment_out="$comment_char "
+               else
+                       comment_out=
+               fi
+
+               if test -z "$rebase_root"
+               then
+                       preserve=t
+                       for p in $(git rev-list --parents -1 $sha1 | cut -d' ' -s -f2-)
+                       do
+                               if test -f "$rewritten"/$p
+                               then
+                                       preserve=f
+                               fi
+                       done
+               else
+                       preserve=f
+               fi
+               if test f = "$preserve"
+               then
+                       touch "$rewritten"/$sha1
+                       printf '%s\n' "${comment_out}pick $sha1 $rest" >>"$todo"
+               fi
+       done
+
+       # Watch for commits that been dropped by --cherry-pick
+       mkdir "$dropped"
+       # Save all non-cherry-picked changes
+       git rev-list $revisions --left-right --cherry-pick | \
+               sed -n "s/^>//p" > "$state_dir"/not-cherry-picks
+       # Now all commits and note which ones are missing in
+       # not-cherry-picks and hence being dropped
+       git rev-list $revisions |
+       while read rev
+       do
+               if test -f "$rewritten"/$rev &&
+                  ! sane_grep "$rev" "$state_dir"/not-cherry-picks >/dev/null
+               then
+                       # Use -f2 because if rev-list is telling us this commit is
+                       # not worthwhile, we don't want to track its multiple heads,
+                       # just the history of its first-parent for others that will
+                       # be rebasing on top of it
+                       git rev-list --parents -1 $rev | cut -d' ' -s -f2 > "$dropped"/$rev
+                       sha1=$(git rev-list -1 $rev)
+                       sane_grep -v "^[a-z][a-z]* $sha1" <"$todo" > "${todo}2" ; mv "${todo}2" "$todo"
+                       rm "$rewritten"/$rev
+               fi
+       done
+
+       complete_action
+}