Re: [PATCH] Add check-valgrind

[Mark Wielaard <mark@klomp.org>]

Hi,

On Thu, Mar 25, 2021 at 02:57:17PM +0100, Tom de Vries wrote:
> Add a make target check-valgrind, that applies valgrind to dwz when used in
> the testsuite.
> 
> Any comments?

It might be good to simply always run under valgrind when it is
installed. That makes make check slightly slower by default, but it
makes sure things are always also ran under valgrind if available.

To make sure that a valgrind tool error really causes a check failure use
--error-exitcode=99 (or some other non-zero code):

   --error-exitcode=<number> [default: 0]
       Specifies an alternative exit code to return if Valgrind reported
       any errors in the run. When set to the default value (zero), the
       return value from Valgrind will always be the return value of the
       process being simulated. When set to a nonzero value, that value is
       returned instead, if Valgrind detects any errors. This is useful
       for using Valgrind as part of an automated test suite, since it
       makes it easy to detect test cases for which Valgrind has reported
       errors, just by inspecting return codes.

Cheers,

Mark