From: Eli Schwartz <eschwartz@archlinux.org>
Date: Fri, 10 Feb 2023 05:28:47 +0000 (-0500)
Subject: meson: correctly specify the dependency relationship for playtests
X-Git-Tag: v1.5.5~2^2~44^2~1
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=183a18a45c1d69f8c42b9fcd25e6d28f9b3d75bb;p=thirdparty%2Fzstd.git

meson: correctly specify the dependency relationship for playtests

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
---

diff --git a/build/meson/tests/meson.build b/build/meson/tests/meson.build
index e70b73432..2dd8d1067 100644
--- a/build/meson/tests/meson.build
+++ b/build/meson/tests/meson.build
@@ -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