When t7510 was introduced, the author made sure that a for loop in
a subshell would return with the appropriate error code.

Make sure this is true also the for the first line in each loop, which
was missed.

Signed-off-by: Michael J Gruber <g...@drmicha.warpmail.net>
---
 t/t7510-signed-commit.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/t/t7510-signed-commit.sh b/t/t7510-signed-commit.sh
index 5ddac1a..a5ba48e 100755
--- a/t/t7510-signed-commit.sh
+++ b/t/t7510-signed-commit.sh
@@ -49,7 +49,7 @@ test_expect_success GPG 'show signatures' '
        (
                for commit in initial second merge fourth-signed fifth-signed 
sixth-signed master
                do
-                       git show --pretty=short --show-signature $commit 
>actual &&
+                       git show --pretty=short --show-signature $commit 
>actual || exit 1
                        grep "Good signature from" actual || exit 1
                        ! grep "BAD signature from" actual || exit 1
                        echo $commit OK
@@ -58,7 +58,7 @@ test_expect_success GPG 'show signatures' '
        (
                for commit in merge^2 fourth-unsigned sixth-unsigned 
seventh-unsigned
                do
-                       git show --pretty=short --show-signature $commit 
>actual &&
+                       git show --pretty=short --show-signature $commit 
>actual || exit 1
                        grep "Good signature from" actual && exit 1
                        ! grep "BAD signature from" actual || exit 1
                        echo $commit OK
-- 
2.0.0.426.g37dbf84

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to