On Thu, Oct 6, 2011 at 1:02 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > I think do_recursive_merge() would die() when the merge cannot even start > (i.e. the local changes after the cherry-pick exits are the ones from the > time before such a failed cherry-pick started), but I suspect that the > other codepath uses try_merge_command() to drive strategies other than > recursive and it does not die() there in such a case. Can you make sure > this patch is sufficient in such a case as well? Why does recursive die with a dirty tree but not other strategies? For that matter, why does revert.c have it's own implementation of recursive instead of just calling try_merge_command("recursive", ...)? j. -- 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