From 6bd5646f1b865a76304a67e03a6161afcfef293f Mon Sep 17 00:00:00 2001 From: Behdad Esfahbod Date: Wed, 9 Jul 2014 17:07:06 -0400 Subject: [PATCH] [tests] Remove bash'ish Apparently on travis-ci, bash is linked to dash, which doesn't understand "let". Failing tests were not being noticed. See eg: https://travis-ci.org/behdad/harfbuzz/jobs/29544211 Don't rely on bash. --- test/shaping/run-tests.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/shaping/run-tests.sh b/test/shaping/run-tests.sh index 4f34f940a..a2cdf32ac 100755 --- a/test/shaping/run-tests.sh +++ b/test/shaping/run-tests.sh @@ -1,4 +1,4 @@ -#!/bin/bash +#!/bin/sh test "x$srcdir" = x && srcdir=. test "x$builddir" = x && builddir=. @@ -21,7 +21,7 @@ for f in "$@"; do if ! test "x$glyphs" = "x$glyphs_expected"; then echo "Actual: $glyphs" >&2 echo "Expected: $glyphs_expected" >&2 - let fails=$fails+1 + fails=$((fails+1)) fi done < "$f" done