author | Junio C Hamano <gitster@pobox.com> | |
Sat, 12 Dec 2009 07:53:41 +0000 (23:53 -0800) | ||
committer | Junio C Hamano <gitster@pobox.com> | |
Sat, 12 Dec 2009 09:22:10 +0000 (01:22 -0800) | ||
commit | 3c5884536563518ce6cd4dc782b0ebb670bf3b6d | |
tree | 2b6aa09fe2d8eb5e2f57e9517c327d0dd50b677a | tree | snapshot |
parent | dd20f8af1ae54773569b78b1b71d1ea663705d2c | commit | diff |
status/commit: do not suggest "reset HEAD <path>" while merging
Suggesting "'reset HEAD <path>' to unstage" is dead wrong if we are about
to record a merge commit. For either an unmerged path (i.e. with
unresolved conflicts), or an updated path, it would result in discarding
what the other branch did.
Note that we do not do anything special in a case where we are amending a
merge. The user is making an evil merge starting from an already
committed merge, and running "reset HEAD <path>" is the right way to get
rid of the local edit that has been added to the index.
Once "reset --unresolve <path>" becomes available, we might want to
suggest it for a merged path that has unresolve information, but until
then, just remove the incorrect advice.
We might also want to suggest "checkout --conflict <path>" to revert the
file in the work tree to the state of failed automerge for an unmerged
path, but we never did that, and this commit does not change that.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Suggesting "'reset HEAD <path>' to unstage" is dead wrong if we are about
to record a merge commit. For either an unmerged path (i.e. with
unresolved conflicts), or an updated path, it would result in discarding
what the other branch did.
Note that we do not do anything special in a case where we are amending a
merge. The user is making an evil merge starting from an already
committed merge, and running "reset HEAD <path>" is the right way to get
rid of the local edit that has been added to the index.
Once "reset --unresolve <path>" becomes available, we might want to
suggest it for a merged path that has unresolve information, but until
then, just remove the incorrect advice.
We might also want to suggest "checkout --conflict <path>" to revert the
file in the work tree to the state of failed automerge for an unmerged
path, but we never did that, and this commit does not change that.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin-commit.c | diff | blob | history | |
t/t7060-wtstatus.sh | diff | blob | history | |
wt-status.c | diff | blob | history | |
wt-status.h | diff | blob | history |