A colleague of mine who is well-intentioned and fairly knowledgeable about git recently caused havoc with our repository while merging changes on the main line branch into a feature branch. The reason is that, along the way, he tried to use "git stash" while the merge was pending. A few commands later, this severely polluted the history of all the files involved.
The command sequence started as follows:
$ git checkout feature-branch $ git merge mainline-branch (conflicts; merge pending) $ git mergetool (resolve all conflicts; takes a while as there are many) $ make && make testsuite (oops, tests don't pass)
At this point, the user notices that the fix to the tests is in the next commit on mainline-branch, which wasn't present when it was first merged. Not wanting to have a commit in the history whose tests do not pass, and not wanting to redo all the conflict resolution work involved in starting over with a different merge parent, he tries to cherry pick it:
$ git fetch $ git cherry-pick commit-with-fix (error: can't cherry pick while merge pending)
Ok, he thinks, git stash has worked in the past as a way to temporarily move aside some work in progress. So:
$ git stash $ git cherry-pick commit-with-fix $ git stash pop
All seems well, so:
$ git commit
Unfortunately, this does *not* do what was intended. You see, unbeknownst to this developer, git stash does not save the MERGE_HEAD! So the commit shows up as an enormous single-parent commit on top of feature-branch, referring to none of the merged commits from mainline-branch in its ancestry. Consequently, "git annotate" and friends regard all the merged changes as having been made by the developer alone and all at once, distorting the file histories to the point of being nearly useless. Furthermore, git still doesn't think mainline-branch has been merged, so there is more fun to be had sorting that out when the next merge from mainline happens.
We are now in the middle of an expensive and risky project to repair the damage, since people pulled and pushed afterward, before the problem was noticed.
I suggest that this problem could easily have been avoided if "git stash" refused to run with a pending merge (present MERGE_HEAD file), since this is crucial repository state that it does not save. This seems similar to what "git cherry-pick" does.
-Scott
