From f84df81f654aeb0ac4582e0b3be162cba6ea5232 Mon Sep 17 00:00:00 2001 From: Ramsay Jones Date: Tue, 28 Apr 2015 13:04:44 +0100 Subject: [PATCH 1/2] t7502-commit.sh: fix a broken and-chain Signed-off-by: Ramsay Jones Signed-off-by: Junio C Hamano --- t/t7502-commit.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/t/t7502-commit.sh b/t/t7502-commit.sh index 051489ea33..2e0d557243 100755 --- a/t/t7502-commit.sh +++ b/t/t7502-commit.sh @@ -370,7 +370,7 @@ exit 0 EOF test_expect_success !AUTOIDENT 'do not fire editor when committer is bogus' ' - >.git/result + >.git/result && >expect && echo >>negative && From 92b269f5c5e5c1d51e9318003caf8ae9d076b4e9 Mon Sep 17 00:00:00 2001 From: Jeff King Date: Wed, 22 Apr 2015 16:09:57 -0400 Subject: [PATCH 2/2] test-lib: turn on GIT_TEST_CHAIN_LINT by default Now that the feature has had time to prove itself, and any topics in flight have had a chance to clean up any broken &&-chains, we can flip this feature on by default. This makes one less thing submitters need to configure or check before sending their patches. Signed-off-by: Jeff King Signed-off-by: Junio C Hamano --- t/test-lib.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/t/test-lib.sh b/t/test-lib.sh index 50b3d3f45d..281c1eca09 100644 --- a/t/test-lib.sh +++ b/t/test-lib.sh @@ -531,7 +531,7 @@ test_run_ () { test_cleanup=: expecting_failure=$2 - if test "${GIT_TEST_CHAIN_LINT:-0}" != 0; then + if test "${GIT_TEST_CHAIN_LINT:-1}" != 0; then # 117 is magic because it is unlikely to match the exit # code of other programs test_eval_ "(exit 117) && $1"