Make the output TAP compliant for tests skipped on request (GIT_SKIP_TESTS). Signed-off-by: Michael J Gruber <git@xxxxxxxxxxxxxxxxxxxx> --- v2 has a typo fix in the commit message and the spelling of "skip" as suggested by AAB. I think I've read it differently in the RFC, but then I was also confused about the "#" in the RFC, and following the "reference" implementation (as per AAB) should be the best approach. t/test-lib.sh | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/t/test-lib.sh b/t/test-lib.sh index bc06564..db8371c 100644 --- a/t/test-lib.sh +++ b/t/test-lib.sh @@ -396,7 +396,7 @@ test_skip () { case "$to_skip" in t) say_color skip >&3 "skipping test: $@" - say_color skip "ok $test_count: # skip $1" + say_color skip "ok $test_count # skip $1" : true ;; *) @@ -833,7 +833,7 @@ do case "$this_test" in $skp) say_color skip >&3 "skipping test $this_test altogether" - say_color skip "skip all tests in $this_test" + skip_all="skip all tests in $this_test" test_done esac done -- 1.7.2.rc1.212.g850a -- 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