aboutsummaryrefslogtreecommitdiff
path: root/t/Makefile
diff options
context:
space:
mode:
authorJens Lehmann <Jens.Lehmann@web.de>2014-07-09 21:34:12 +0200
committerJunio C Hamano <gitster@pobox.com>2014-07-10 13:04:34 -0700
commitcd78cea29d5476c6178cd49c08d10566f5bf0d4f (patch)
tree5ec158fc906d264b5a9db3e8d5dde2d508f2ea19 /t/Makefile
parent6f92e5ff3cdc813de8ef5327fd4bad492fb7d6c9 (diff)
downloadgit-cd78cea29d5476c6178cd49c08d10566f5bf0d4f.tar.gz
git-cd78cea29d5476c6178cd49c08d10566f5bf0d4f.tar.xz
t/Makefile: check helper scripts for non-portable shell commands too
Currently only the "t[0-9][0-9][0-9][0-9]-*.sh" scripts are tested for shell incompatibilities using the check-non-portable-shell.pl script. This makes it easy to miss non-POSIX constructs added to one of the t/*lib*.sh helper scripts, as they aren't automatically detected. Fix that by adding a THELPERS variable containing all shell scripts that aren't tests and add these to the "test-lint-shell-syntax" target too. Signed-off-by: Jens Lehmann <Jens.Lehmann@web.de> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/Makefile')
-rw-r--r--t/Makefile3
1 files changed, 2 insertions, 1 deletions
diff --git a/t/Makefile b/t/Makefile
index 8fd1a7235..7fa66928b 100644
--- a/t/Makefile
+++ b/t/Makefile
@@ -29,6 +29,7 @@ TEST_RESULTS_DIRECTORY_SQ = $(subst ','\'',$(TEST_RESULTS_DIRECTORY))
T = $(sort $(wildcard t[0-9][0-9][0-9][0-9]-*.sh))
TSVN = $(sort $(wildcard t91[0-9][0-9]-*.sh))
TGITWEB = $(sort $(wildcard t95[0-9][0-9]-*.sh))
+THELPERS = $(sort $(filter-out $(T),$(wildcard *.sh)))
all: $(DEFAULT_TEST_TARGET)
@@ -65,7 +66,7 @@ test-lint-executable:
echo >&2 "non-executable tests:" $$bad; exit 1; }
test-lint-shell-syntax:
- @'$(PERL_PATH_SQ)' check-non-portable-shell.pl $(T)
+ @'$(PERL_PATH_SQ)' check-non-portable-shell.pl $(T) $(THELPERS)
aggregate-results-and-cleanup: $(T)
$(MAKE) aggregate-results