From f75cb3c5d52a7cae7ad849660b495966bb07baac Mon Sep 17 00:00:00 2001 From: Sergei Zimmerman <145775305+xokdvium@users.noreply.github.com> Date: Wed, 8 Jan 2025 19:18:22 +0300 Subject: [PATCH] cbmc: use versionCheckHook instead of testers.testVersion There's no reason to use passthru.tests for this and manual recommends to do this by default. --- pkgs/by-name/cb/cbmc/package.nix | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/pkgs/by-name/cb/cbmc/package.nix b/pkgs/by-name/cb/cbmc/package.nix index 70eb5fa01871..fe86380d9b6f 100644 --- a/pkgs/by-name/cb/cbmc/package.nix +++ b/pkgs/by-name/cb/cbmc/package.nix @@ -2,10 +2,8 @@ lib, stdenv, fetchFromGitHub, - testers, bison, cadical, - cbmc, cmake, flex, makeWrapper, @@ -14,6 +12,7 @@ cudd, nix-update-script, fetchpatch, + versionCheckHook, }: stdenv.mkDerivation (finalAttrs: { @@ -109,10 +108,12 @@ stdenv.mkDerivation (finalAttrs: { "-Dsat_impl=cadical" ]; - passthru.tests.version = testers.testVersion { - package = cbmc; - command = "cbmc --version"; - }; + nativeInstallCheckInputs = [ + versionCheckHook + ]; + doInstallCheck = true; + versionCheckProgram = "${placeholder "out"}/bin/cbmc"; + versionCheckProgramArg = "--version"; passthru.updateScript = nix-update-script { extraArgs = [