make: add maintainer-check-valgrind make target

abrt / satyr

Automatic problem management with anonymous reports

GNU General Public License v2.0

19 stars 22 forks source link

make: add maintainer-check-valgrind make target #305

Closed michalfabik closed 4 years ago

michalfabik commented 4 years ago

The first commit only applies to the tests that are still not dealt with in Ernestas's incomplete PR #298.
The second commit adds the check-valgrind make target that applies to the changes in #298. Once #298 is finished, the changes in the first commit can be removed again.

Signed-off-by: Michal Fabik mfabik@redhat.com

mzidek-gh commented 4 years ago

So, should we first review and push the #298 and then this?

This patch does not seem to break anything, but the make check-valgrind does not do anything at this point. Do you plan to add new patch to enable the memcheck for the tests subdirectory?