On Mon, Dec 30, 2013 at 10:51:25AM -0800, Jonathan Nieder wrote: > I think it can be better, since the commit message left me scratching > my head while the patch itself seems pretty simple. How about > something like the following? I am fine with that format, though... > Analysis and fix: > > These scratch areas for sub-tests should be under the t0000 > trash directory, but because the TEST_OUTPUT_DIRECTORY > setting from the toplevel test leaks into the environment > they are created under the toplevel output directory (typically > t/) instead. Because some of the sub-tests simulate failures, > their trash directories are kept around. This is not exactly true. The TEST_OUTPUT_DIRECTORY setting does not leak. t0000 sets $TEST_DIRECTORY (which it must, so the sub-scripts can find test-lib.sh and friends), and then TEST_OUTPUT_DIRECTORY uses that as a default if it is not explicitly set. The rest of your rewrite looks correct. -Peff -- 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