Send patches - preferably formatted by git format-patch - to patches at archlinux32 dot org.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEli Schwartz <eschwartz@archlinux.org>2019-11-25 19:26:44 -0500
committerAllan McRae <allan@archlinux.org>2019-11-26 11:05:44 +1000
commit3073752bcd9718b243661dd727e8f8bc18035938 (patch)
tree93c96808f33533f7f1f9959951a5b0cd8d2532e0
parent49a8e90bc78c17600eb4903ba00be26341df9161 (diff)
pactest: add environment option to run tests with valgrind
In autotools, if we wanted to run tests with valgrind, we used some Make magic which passed arguments to pactest.py, but that doesn't work in meson, because all arguments are encoded at configure time. Instead, let's short-circuit the build runner logic entirely, and teach pactest to default to running valgrind, when it detects an environment variable set independent of the build system. To run the tests with valgrind, we can now use: PACTEST_VALGRIND=1 meson test -C builddir/ or PACTEST_VALGRIND=1 make check It is also possible, but confusing/inconsistent, to use make check PY_LOG_FLAGS=--valgrind We *could* add a meson option -Dvalgrind=true, but that is annoying to reconfigure between test runs, and overall the consensus is it seems simpler to opt in each time we want to run valgrind, as was already the case. Signed-off-by: Eli Schwartz <eschwartz@archlinux.org> Signed-off-by: Allan McRae <allan@archlinux.org>
-rwxr-xr-xtest/pacman/pactest.py2
1 files changed, 1 insertions, 1 deletions
diff --git a/test/pacman/pactest.py b/test/pacman/pactest.py
index a5891d17..3414f4cc 100755
--- a/test/pacman/pactest.py
+++ b/test/pacman/pactest.py
@@ -96,7 +96,7 @@ def create_parser():
dest = "gdb", default = False,
help = "use gdb while calling pacman")
parser.add_option("--valgrind", action = "store_true",
- dest = "valgrind", default = False,
+ dest = "valgrind", default = os.getenv('PACTEST_VALGRIND'),
help = "use valgrind while calling pacman")
parser.add_option("--manual-confirm", action = "store_true",
dest = "manualconfirm", default = False,