From be772126ca5d67b99c76ffb719e5c4b66cc37577 Mon Sep 17 00:00:00 2001 From: Sergei Zimmerman Date: Sat, 10 May 2025 19:19:30 +0000 Subject: [PATCH] halide: use ctestCheckHook --- pkgs/development/compilers/halide/default.nix | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/pkgs/development/compilers/halide/default.nix b/pkgs/development/compilers/halide/default.nix index 3ce25abaf53c..821459bacdae 100644 --- a/pkgs/development/compilers/halide/default.nix +++ b/pkgs/development/compilers/halide/default.nix @@ -22,6 +22,7 @@ wasmSupport ? false, wabt, doCheck ? true, + ctestCheckHook, }: assert blas.implementation == "openblas" && lapack.implementation == "openblas"; @@ -102,15 +103,9 @@ stdenv.mkDerivation (finalAttrs: { "correctness_cross_compilation" "correctness_simd_op_check_hvx" ]; - # ninja's setup-hook doesn't let us specify custom flags for the checkPhase, see - # https://discourse.nixos.org/t/building-only-specific-targets-with-cmake/31545/4 - # so we resort to overriding the whole checkPhase + dontUseNinjaCheck = true; - checkPhase = '' - runHook preCheck - ctest --exclude-regex '^(${lib.strings.concatStringsSep "|" finalAttrs.disabledTests})$' - runHook postCheck - ''; + nativeCheckInputs = [ ctestCheckHook ]; postInstall = lib.optionalString pythonSupport ''