On Sat, Dec 28, 2013 at 12:51:04PM -0800, Jonathan Nieder wrote:
Jeff King wrote:
I am not _that_ bothered by the known breakage, but AFAICT there is
zero benefit to keeping this redundant test.
Devil's advocate: it ensures that anyone wrapping git's tests (like
the old smoketest infrastructure experiment) is able to handle an
expected failure.
Thanks. One of the things I love about open source is that as soon as I
say I can't see how..., the answer is crowd-sourced for me. :)
That being said, even if the test has a non-zero possible value...
But in practice I don't mind the behavior before or after this patch.
If the test harness is that broken, we'll know. And people writing
code that wraps git's tests can write their own custom sanity-checks.
...I think for these reasons that the value is smaller than the
disruption caused by the test, and the patch is a net win.
-Peff
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
