From c26cfcbf950d2e9c31a2707ea6fa32b397c6149a Mon Sep 17 00:00:00 2001 From: Jared Baur Date: Fri, 3 Jan 2025 12:36:54 -0800 Subject: [PATCH] nixos-firewall-tool: fix cross-compilation The shellcheck dependency should be placed in `nativeCheckInputs` to ensure it is obtained from the correct package-set. --- pkgs/by-name/ni/nixos-firewall-tool/package.nix | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/pkgs/by-name/ni/nixos-firewall-tool/package.nix b/pkgs/by-name/ni/nixos-firewall-tool/package.nix index a896109aa6a4..9c1aa059d32d 100644 --- a/pkgs/by-name/ni/nixos-firewall-tool/package.nix +++ b/pkgs/by-name/ni/nixos-firewall-tool/package.nix @@ -6,7 +6,7 @@ shellcheck-minimal, }: -stdenvNoCC.mkDerivation { +stdenvNoCC.mkDerivation (finalAttrs: { name = "nixos-firewall-tool"; src = builtins.filterSource (name: _: !(lib.hasSuffix ".nix" name)) ./.; @@ -14,6 +14,7 @@ stdenvNoCC.mkDerivation { strictDeps = true; buildInputs = [ bash ]; nativeBuildInputs = [ installShellFiles ]; + nativeCheckInputs = lib.optionals finalAttrs.doCheck [ shellcheck-minimal ]; postPatch = '' patchShebangs --host nixos-firewall-tool @@ -30,7 +31,7 @@ stdenvNoCC.mkDerivation { lib.meta.availableOn stdenvNoCC.buildPlatform shellcheck-minimal.compiler && (builtins.tryEval shellcheck-minimal.compiler.outPath).success; checkPhase = '' - ${lib.getExe shellcheck-minimal} nixos-firewall-tool + shellcheck nixos-firewall-tool ''; meta = with lib; { @@ -44,4 +45,4 @@ stdenvNoCC.mkDerivation { platforms = platforms.linux; mainProgram = "nixos-firewall-tool"; }; -} +})