tests: turn on test-lint by default
authorJeff King <peff@peff.net>
Thu, 3 Jan 2013 07:17:51 +0000 (02:17 -0500)
committerJunio C Hamano <gitster@pobox.com>
Thu, 3 Jan 2013 16:03:46 +0000 (08:03 -0800)
The test Makefile knows about a few "lint" checks for common
errors. However, they are not enabled as part of "make test"
by default, which means that many people do not bother
running them. Since they are both quick to run and accurate
(i.e., no false positives), there should be no harm in
turning them on and helping submitters catch errors earlier.

We could just set:

  TEST_LINT = test-lint

to enable all tests. But that would be unnecessarily
annoying later on if we add slower or less accurate tests
that should not be part of the default. Instead, we name the
tests individually.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/Makefile

index 3025418ff57c8c16cd11dd0db4f9d442743118fa..5c6de8169bfc02fc280d2dcb8cf34a912f0b0f6a 100644 (file)
@@ -13,6 +13,7 @@ TAR ?= $(TAR)
 RM ?= rm -f
 PROVE ?= prove
 DEFAULT_TEST_TARGET ?= test
+TEST_LINT ?= test-lint-duplicates test-lint-executable
 
 # Shell quote;
 SHELL_PATH_SQ = $(subst ','\'',$(SHELL_PATH))