diff options
author | sternenseemann <sternenseemann@systemli.org> | 2023-09-03 23:01:22 +0200 |
---|---|---|
committer | sternenseemann <sternenseemann@systemli.org> | 2023-09-09 17:55:37 +0200 |
commit | bd374243c08040fba0897727d05e05f08145fdf2 (patch) | |
tree | 2ea5dde827bc31fab90c3866c763ef0c2309b576 | |
parent | 9f48070e89e75281391ad80816c0aaf74f85e3d2 (diff) |
npmHooks: use adjacent packages, not buildPackages
Hooks are essentially implemented as special shell packages that run on their respective host platform. When they are used, they appear as nativeBuildInputs (as they need to be executed as part of the build of a package using them) so are taken from buildPackages relative to the derivation using them. Since the override in buildNpmPackage nullifies splicing, we take npmHooks from buildPackages manually. Fixes pkgsCross.ghcjs.buildPackages.emscripten and thus pkgsCross.ghcjs.haskellPackages.ghc.
-rw-r--r-- | pkgs/build-support/node/build-npm-package/default.nix | 9 | ||||
-rw-r--r-- | pkgs/build-support/node/build-npm-package/hooks/default.nix | 23 |
2 files changed, 23 insertions, 9 deletions
diff --git a/pkgs/build-support/node/build-npm-package/default.nix b/pkgs/build-support/node/build-npm-package/default.nix index 0a988cbbaf3f2..7cfc0e9f9c0a5 100644 --- a/pkgs/build-support/node/build-npm-package/default.nix +++ b/pkgs/build-support/node/build-npm-package/default.nix @@ -1,4 +1,4 @@ -{ lib, stdenv, fetchNpmDeps, npmHooks, nodejs }: +{ lib, stdenv, fetchNpmDeps, buildPackages, nodejs }: { name ? "${args.pname}-${args.version}" , src ? null @@ -44,7 +44,12 @@ let hash = npmDepsHash; }; - inherit (npmHooks.override { inherit nodejs; }) npmConfigHook npmBuildHook npmInstallHook; + # .override {} negates splicing, so we need to use buildPackages explicitly + npmHooks = buildPackages.npmHooks.override { + inherit nodejs; + }; + + inherit (npmHooks) npmConfigHook npmBuildHook npmInstallHook; in stdenv.mkDerivation (args // { inherit npmDeps npmBuildScript; diff --git a/pkgs/build-support/node/build-npm-package/hooks/default.nix b/pkgs/build-support/node/build-npm-package/hooks/default.nix index 3f2b0adf1668d..36f0319e3d230 100644 --- a/pkgs/build-support/node/build-npm-package/hooks/default.nix +++ b/pkgs/build-support/node/build-npm-package/hooks/default.nix @@ -1,4 +1,13 @@ -{ lib, makeSetupHook, nodejs, srcOnly, buildPackages, makeWrapper }: +{ lib +, srcOnly +, makeSetupHook +, makeWrapper +, nodejs +, jq +, prefetch-npm-deps +, diffutils +, installShellFiles +}: { npmConfigHook = makeSetupHook @@ -6,13 +15,13 @@ name = "npm-config-hook"; substitutions = { nodeSrc = srcOnly nodejs; - nodeGyp = "${buildPackages.nodejs}/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js"; + nodeGyp = "${nodejs}/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js"; # Specify `diff`, `jq`, and `prefetch-npm-deps` by abspath to ensure that the user's build # inputs do not cause us to find the wrong binaries. - diff = "${buildPackages.diffutils}/bin/diff"; - jq = "${buildPackages.jq}/bin/jq"; - prefetchNpmDeps = "${buildPackages.prefetch-npm-deps}/bin/prefetch-npm-deps"; + diff = "${diffutils}/bin/diff"; + jq = "${jq}/bin/jq"; + prefetchNpmDeps = "${prefetch-npm-deps}/bin/prefetch-npm-deps"; nodeVersion = nodejs.version; nodeVersionMajor = lib.versions.major nodejs.version; @@ -27,13 +36,13 @@ npmInstallHook = makeSetupHook { name = "npm-install-hook"; - propagatedBuildInputs = with buildPackages; [ + propagatedBuildInputs = [ installShellFiles makeWrapper ]; substitutions = { hostNode = "${nodejs}/bin/node"; - jq = "${buildPackages.jq}/bin/jq"; + jq = "${jq}/bin/jq"; }; } ./npm-install-hook.sh; } |