From 183a18a45c1d69f8c42b9fcd25e6d28f9b3d75bb Mon Sep 17 00:00:00 2001 From: Eli Schwartz Date: Fri, 10 Feb 2023 00:28:47 -0500 Subject: [PATCH] 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 --- build/meson/tests/meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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