This is the mail archive of the
elfutils-devel@sourceware.org
mailing list for the elfutils project.
Re: [Patch] Add configure --enable-valgrind option to run all tests under valgrind.
- From: Roland McGrath <roland at hack dot frob dot com>
- To: elfutils-devel at lists dot fedorahosted dot org
- Date: Tue, 09 Oct 2012 14:48:21 -0700
- Subject: Re: [Patch] Add configure --enable-valgrind option to run all tests under valgrind.
I didn't get to this before you committed it, and that's OK.
I had some tiny style quibbles. Mostly it just seemed to me
that at least the lower layers of this where it's really just
a generic "command prefix before running a binary" should be
named generically. That could be plumbed all the way up to
configure, as --enable-tests-prefix="...". Then you can add
--enable-tests-valgrind as a shorthand for a particular value
of the prefix.
The log entry mentions valgrind-wrapper.sh but no such file nor
use of it appears in the patch.