From f7025e2f731a29ab44309c01c05fef646f170ceb Mon Sep 17 00:00:00 2001 From: Uli Schlachter Date: Sun, 1 Jan 2017 14:51:39 +0100 Subject: [PATCH] Fix tests/run.sh Commit a944636c0281314ef "bashified" tests/run.sh for some reason. Afterwards, commit 4abd820051d3e4b09d fixed some of the fall-out. However, there is still a problem left. We have "set -e" in this script. Thus, whenever some command exits with status 0, the script abort. When the variable errors is zero/unset, the command "((errors++))" has exit status zero. Thus, this instruction caused the shell script to abort. This was not intended. Fix this by using "((++errors))" instead. Signed-off-by: Uli Schlachter --- tests/run.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/run.sh b/tests/run.sh index 24caf274..64f3f941 100755 --- a/tests/run.sh +++ b/tests/run.sh @@ -228,7 +228,7 @@ for f in $tests; do grep -q -E '[Ee]rror|assertion failed' "$awesome_log"; then echo "===> ERROR running $f <===" grep --color -o --binary-files=text -E '.*[Ee]rror.*|.*assertion failed.*' "$awesome_log" || true - ((errors++)) + ((++errors)) fi done