Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [RFC/PATCH] fail pull/merge early in the middle of conflicted merge

From: Horst H. von Brand <hidden>
Date: 2016-06-15 22:42:47

Andreas Ericsson [off-list ref] wrote:
Junio C Hamano wrote:
quoted
"Shawn O. Pearce" [off-list ref] writes:
quoted
Junio C Hamano [off-list ref] wrote:
quoted
After a pull that results in a conflicted merge, a new user
often tries another "git pull" in desperation.  When the index
is unmerged, merge backends correctly bail out without touching
either index nor the working tree, so this does not make the
wound any worse.
I've seen this many times too.  I don't understand why users cannot
read output messages and realize the current command failed, but
they don't.  *sigh*
That is not user's fault.  Tools should not make things worse
when run more than once after they failed, and we do not either,
so it is not a stupid behaviour on the user's part to do that.
We just need to make sure that it is more clear to the user that
pull after a conflicted pull is futile, which is what this patch
does.
"Pulling is futile. Nothing will be assimilated" ?
:-)

Been there, done that. The messages git gives don't give a (at this point
probably panicked) newbie any clue what to try next, or how to go back.

Perhaps best would be to offer some way to undo all and then try again?

Also, if a manual fix of the merge is warranted, the message doesn't give
clear directions on what to do next. Sure, you edit the offending files to
your liking, then what? "git commit" shows a scary commit message, you
abort the edit and the merge is commited anyway.
-- 
Dr. Horst H. von Brand                   User #22616 counter.li.org
Departamento de Informatica                    Fono: +56 32 2654431
Universidad Tecnica Federico Santa Maria             +56 32 2654239
Casilla 110-V, Valparaiso, Chile               Fax:  +56 32 2797513
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help