Matthieu Moy <Matthieu.Moy@xxxxxxx> writes:
> I'm also concerned about the possibility of stash/unstash to fail:
> that means the user has to deal with two kinds of conflicts (rebase
> can conflict, unstash can also), which can be confusing.
Unstash should be invoked automatically _only_ after rebase
completes, so I do not forsee such confusion.
But the trouble I have with the auto unstashing is more at the
conceptual and workflow level. You start rebasing a branch, and
your work tree is dirty. What branch should the local
modification belong to?
Logically, it is with the branch you were on when you typed "git
rebase" command.
But when "rebase" successfully concludes (either with or without
manual fix-ups), you can be either on your original branch (if
you said "git rebase base") or something totally unrelated (if
you said "git rebase base other"). Currently we auto-unstash in
both cases. I _think_ the former case should auto-unstash, but
the latter shouldn't.
However, this auto-stash uses the new "git stash create" command
that does not update the reflog for "refs/stash", specifically
so that the auto-stashing does _not_ interfere with the normal
stash the end user uses. Which means that it is a bit
cumbersome to give the autostashed state back to the user if we
do _not_ unstash upon completion of rebase.
Perhaps a good compromise would be to
* not to do the autostash, but fail as we have always done,
when "git rebase base other" form was used, first instructing
rebase to switch to another branch;
* autostash when "git rebase base" form was used, and auto
unstash upon completion.
> But the current behavior can be greatly improved by just making the
> error message better. Currently, I have this:
>
> $ git rebase branch
> foo.txt: needs update
> $ _
>
> I'd prefer something like
>
> $ git rebase branch
> cannot rebase: the working tree is not clean.
> foo.txt: Changed but not updated
> Commit your changes, or put them appart with "git stash" and retry.
> $ _
You forgot 'needs merge' case, so that would not fly very well,
but something like this might be a good starting point.
---
git-rebase.sh | 5 ++++-
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/git-rebase.sh b/git-rebase.sh
index 058fcac..4f8aeb9 100755
--- a/git-rebase.sh
+++ b/git-rebase.sh
@@ -252,7 +252,10 @@ else
fi
# The tree must be really really clean.
-git update-index --refresh || exit
+o=$(git update-index -q --refresh) || {
+ printf "cannot rebase: the work tree is not clean.\n%s\n" "$o"
+ exit 1
+}
diff=$(git diff-index --cached --name-status -r HEAD)
case "$diff" in
?*) echo "cannot rebase: your index is not up-to-date"
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
