Make run-testsuite recognize NO_VALGRIND environment variable to work around a
authorMatt McCutchen <matt@mattmccutchen.net>
Wed, 16 Jul 2008 20:11:12 +0000 (16:11 -0400)
committerMatt McCutchen <matt@mattmccutchen.net>
Wed, 16 Jul 2008 20:11:12 +0000 (16:11 -0400)
Valgrind bug:

https://bugzilla.redhat.com/show_bug.cgi?id=455644

run-testsuite

index 0c563c2..116d2c9 100755 (executable)
@@ -2,9 +2,15 @@
 
 bad=
 
+if [ "$NO_VALGRIND" ]; then
+       cmd=(./testsuite)
+else
+       cmd=(valgrind --error-exitcode=1 --leak-check=full ./testsuite)
+fi
+
 set -o pipefail
-if ! valgrind --error-exitcode=1 --leak-check=full \
-       ./testsuite 2>&1 >testsuite.out | tee testsuite.err; then
+# Stdout goes directly to testsuite.out; stderr goes down the pipe.
+if ! "${cmd[@]}" 2>&1 >testsuite.out | tee testsuite.err; then
        echo >&2 'Memory errors!'
        bad=1
 fi