[Libguestfs] [PATCH nbdkit 2/4] valgrind: Add --show-leak-kinds=all and comprehensive list of suppressions.
Eric Blake
eblake at redhat.com
Tue Dec 4 04:06:17 UTC 2018
On 12/3/18 12:55 PM, Eric Blake wrote:
> On 12/2/18 10:39 AM, Richard W.M. Jones wrote:
>> By default valgrind suppresses many leaks. I'm not even sure exactly
>> how it decides which ones to suppress, but certainly global variables
>> pointing to malloc’d data are suppressed, which is not useful
>> behaviour.
>
>
>> +include $(top_srcdir)/common-rules.mk
>> +
>> +suppressions_files = $(wildcard *.suppressions)
>
> A GNU make-ism - but you already mention requiring GNU make in README.
> Should we make ./configure error out hard if $MAKE is not GNU Make,
> rather than risking someone getting 80% though a build on BSD make and
> then choking when it gets here?
Worse, it fails in VPATH builds. It resulted in 'make distcheck'
hanging on a 'cat > valgrind.suppressions' command, because the wildcard
didn't find anything in $(builddir), when it should have been looking in
$(srcdir).
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
More information about the Libguestfs
mailing list