]> git.ipfire.org Git - thirdparty/zstd.git/commitdiff
meson: correctly specify the dependency relationship for playtests
authorEli Schwartz <eschwartz@archlinux.org>
Fri, 10 Feb 2023 05:28:47 +0000 (00:28 -0500)
committerEli Schwartz <eschwartz@archlinux.org>
Fri, 10 Feb 2023 05:33:07 +0000 (00:33 -0500)
It depends on the zstd program being built, and passes it as an env
variable. Just like datagen. But for datagen, we explicitly depend on
it, while for zstd, we assume it's built as part of "all".

This can be wrong in two cases:
- when running individual tests, meson can (re)build just what is needed
  for that one test
- a later patch will handle building zstd but not by default

build/meson/tests/meson.build

index e70b73432c5d1a3091754410c7bdc31669c74221..2dd8d1067e08b3b4f44eb450d510158c6ad1c527 100644 (file)
@@ -162,7 +162,7 @@ if host_machine_os != os_windows
       playTests_sh,
       args: opt,
       env: ['ZSTD_BIN=' + zstd.full_path(), 'DATAGEN_BIN=./datagen'],
-      depends: [datagen],
+      depends: [datagen, zstd],
       suite: suite,
       workdir: meson.current_build_dir(),
       timeout: 2800) # Timeout should work on HDD drive