On Tue, Jan 27, 2009 at 10:01 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Jay Soffian <jaysoffian@xxxxxxxxx> writes: > >> Also ensure that $dotest is cleaned up if user manages to interupt mailsplit >> while it is processing input. > > I do not think this "Also ensure" part is necessary nor desirable. If > something goes wrong, we'd like to have a way to resurrect some partial > results from the split out result. Hmm, git-am was already nuking $dotest if mailsplit exited non-zero, so I kept that behavior, but also clean-up if user kills git-am while mailsplit is running. I still think it should nuke $dotest if user hits ctrl-c, but I agree if mailsplit exits non-zero it should not (which would be a change from what it does now). > The rest looked fine. Perhaps a re-roll? Sure. 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
