From dd44ccef0da458b682bcb6b866aac1460a5b50a1 Mon Sep 17 00:00:00 2001 From: Pádraig Brady Date: Thu, 20 Dec 2012 00:09:15 +0000 Subject: tests: avoid false positive valgrind failures * init.cfg (require_valgrind_): Check the `true` program, which will check more valgrind failure cases as now detailed in the function comments. --- init.cfg | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/init.cfg b/init.cfg index e853b0a2f..ce5a106de 100644 --- a/init.cfg +++ b/init.cfg @@ -159,10 +159,14 @@ require_strace_() fi } -# Skip the current test if valgrind doesn't work. +# Skip the current test if valgrind doesn't work, +# which could happen if not installed, +# or hasn't support for the built architecture, +# or hasn't appropriate error suppressions installed etc. require_valgrind_() { - valgrind --help >/dev/null || skip_ "requires valgrind" + valgrind --error-exitcode=1 true 2>/dev/null || + skip_ "requires a working valgrind" } require_setfacl_() -- cgit v1.2.3-70-g09d2