]> git.ipfire.org Git - thirdparty/zstd.git/commit
meson: don't require valgrind tests
authorEli Schwartz <eschwartz@archlinux.org>
Wed, 20 Apr 2022 00:58:33 +0000 (20:58 -0400)
committerEli Schwartz <eschwartz@archlinux.org>
Fri, 16 Dec 2022 19:27:46 +0000 (14:27 -0500)
commit26134b4565a85e133f6f77b16b4fe9cd0c530a07
tree8dcae858dafc70d30bbb1fd49a77d70c0b2ffe03
parentef566c8d683212a04174124ef3c0d34cdfb583b5
meson: don't require valgrind tests

It's entirely possible some people don't have valgrind installed, but
still want to run the tests. If they don't have it installed, then they
probably don't intend to run those precise test targets anyway.

Also, this solves an error when running the tests in an automated
environment. The valgrind tests have a hard dependency on behavior such
as `./zstd` erroring out with the message "stdin is a console, aborting"
which does not work if the automated environment doesn't have a console.
As a rough heuristic, automated environments lacking a console will
*probably* also not have valgrind, so avoiding that test definition
neatly sidesteps the issue.

Also, valgrind is not easily installable on macOS, at least homebrew
says it isn't available there. This makes it needlessly hard to
enable the testsuite on macOS.
build/meson/tests/meson.build