Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes: > Hi, > > On Wed, 22 Oct 2008, Jeff King wrote: > >> diff --git a/Makefile b/Makefile >> index d6f3695..68f0172 100644 >> --- a/Makefile >> +++ b/Makefile >> @@ -1343,7 +1345,12 @@ all:: $(TEST_PROGRAMS) >> >> export NO_SVN_TESTS >> >> -test: all >> +valgrind-setup: $(patsubst %.sh,%,$(VALGRIND_SH)) >> + rm -rf t/valgrind/bin >> + mkdir t/valgrind/bin >> + for i in git $(PROGRAMS); do cp test-valgrind t/valgrind/bin/$$i; done > > I wonder if it would not be better to scrap the t/valgrind/ directory and > regenerate it everytime you run a test manually; I'd use "ln" instead of > "cp", and also parse command-list.txt to catch really all of them (even if > a dashed form is used for a builtin by mistake). Going one step further, I wonder if this approach can also be used to catch such a mistake. Install a dashed form that records the fact that it was called when it shouldn't, and by whom. -- 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