On 10/02/2015 03:22 PM, Lukas Slebodnik wrote:
attached patch should fix valgrind issues with whitespace_test
Thank you, Lukas. This seems fine. I have also made another patch which excludes whitespace_test from Valgrind run instead of renaming it. I'm not sure which is better and actually don't quite understand how your version works, but here it is :) Nick
>From 72533b05c2b2bcb90771bf9c91e9dade6e94b579 Mon Sep 17 00:00:00 2001 From: Nikolai Kondrashov <nikolai.kondras...@redhat.com> Date: Fri, 2 Oct 2015 15:54:23 +0300 Subject: [PATCH] CI: Exclude whitespace_test from Valgrind checks Exclude whitespace_test from Valgrind checks in contrib/ci/run to prevent it from failing the tests due to Bash bugs. --- contrib/ci/run | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/contrib/ci/run b/contrib/ci/run index 5f668ff..4d89c8a 100755 --- a/contrib/ci/run +++ b/contrib/ci/run @@ -206,7 +206,7 @@ function build_debug() make-check-wrap -j $CPU_NUM check -- \ libtool --mode=execute \ valgrind-condense 99 \ - '!(*.py|*dlopen-tests)' -- \ + '!(*.py|*dlopen-tests|*/whitespace_test)' -- \ --trace-children=yes \ --trace-children-skip='*/bin/*,*/sbin/*' \ --leak-check=full \ -- 2.5.1
_______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/sssd-devel