On Fri, Oct 15, 2010 at 12:42 PM, Jeff King <peff@xxxxxxxx> wrote: > On Thu, Oct 14, 2010 at 03:37:21PM -0500, Jonathan Nieder wrote: > >> > Oops, good catch. I think we should already catch it, as test_terminal >> > will not be defined at all in the no-tty case. We could print a nicer >> > message, but >> >> I rather meant something like this. >> >> Patch 1 exposes the internal $prereq variable from >> test_expect_(success|failure). Maybe it should be called >> GIT_TEST_something to avoid trampling other programs' namespaces? Not >> sure. >> >> Patch 2 introduces some magic autodetection so people that never run >> tests without -v can still notice the missing TTY prereqs. > > Yeah, that is better, as it will catch the lack of prerequisite even on > systems where the prerequisite is met. > > It seems like a lot of code to catch something small, but on the other > hand, it does seem to be a repeated mistake. I'll probably be re-rolling the push --progress fix series with this and Jeff's. -- Cheers, Ray Chuan -- 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