Jeff King schrieb:
> Why not the much shorter:
>
> tests() {
> ls | sed -n 's/^\(t[0-9][0-9][0-9][0-9]\)-.*\.sh$/\1/p'
> }
> dups=`tests | uniq -d`
>
> instead of the long shell loop?
Because it's faster on Windows ;-) No, seriously, I didn't know about
uniq -d.
Here is a replacement patch.
--- 8< ---
From: Johannes Sixt <j6t@xxxxxxxx>
Subject: [PATCH] t/Makefile: add a check that all test numbers are unique
The test runs only if 'make all' is used. Its purpose is to alert our
valued integrator if different branches are merged that happen to
introduce the same test number.
Signed-off-by: Johannes Sixt <j6t@xxxxxxxx>
---
t/Makefile | 4 ++++
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/t/Makefile b/t/Makefile
index bd09390..ed8281a 100644
--- a/t/Makefile
+++ b/t/Makefile
@@ -17,6 +17,10 @@ T = $(wildcard t[0-9][0-9][0-9][0-9]-*.sh)
TSVN = $(wildcard t91[0-9][0-9]-*.sh)
all: pre-clean
+ @dups=`ls | sed -n 's/^\(t[0-9][0-9][0-9][0-9]\)-.*\.sh$$/\1/p' | \
+ uniq -d` && \
+ test -z "$$dups" || { \
+ echo "duplicate test numbers:" $$dups; exit 1; }
$(MAKE) aggregate-results-and-cleanup
$(T):
--
1.6.4.204.g6aad7
--
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