diff options
author | Will Dietz <w@wdtz.org> | 2017-12-22 16:05:33 -0500 |
---|---|---|
committer | John Ericson <John.Ericson@Obsidian.Systems> | 2018-07-09 11:07:10 -0400 |
commit | 218d4dc154faca85c5e0024c6ab7bf2d1ddac0a5 (patch) | |
tree | 031fea218d440643effc32610e71929c899c040d /pkgs/stdenv | |
parent | 5c271db791c1538c5cbcbb8845e0fa0f1e1abfd8 (diff) | |
download | nixpkgs-218d4dc154faca85c5e0024c6ab7bf2d1ddac0a5.tar nixpkgs-218d4dc154faca85c5e0024c6ab7bf2d1ddac0a5.tar.gz nixpkgs-218d4dc154faca85c5e0024c6ab7bf2d1ddac0a5.tar.bz2 nixpkgs-218d4dc154faca85c5e0024c6ab7bf2d1ddac0a5.tar.lz nixpkgs-218d4dc154faca85c5e0024c6ab7bf2d1ddac0a5.tar.xz nixpkgs-218d4dc154faca85c5e0024c6ab7bf2d1ddac0a5.tar.zst nixpkgs-218d4dc154faca85c5e0024c6ab7bf2d1ddac0a5.zip |
make-derivation: Don't add host-suffix to fixed-output derivations names
Not only does the suffix unnecessarily reduce sharing, but it also breaks unpacker setup hooks (e.g. that of `unzip`) which identify interesting tarballs using the file extension. This also means we can get rid of the splicing hacks for fetchers.
Diffstat (limited to 'pkgs/stdenv')
-rw-r--r-- | pkgs/stdenv/generic/make-derivation.nix | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/pkgs/stdenv/generic/make-derivation.nix b/pkgs/stdenv/generic/make-derivation.nix index dce715a6a53..778f107f71e 100644 --- a/pkgs/stdenv/generic/make-derivation.nix +++ b/pkgs/stdenv/generic/make-derivation.nix @@ -77,8 +77,13 @@ rec { , ... } @ attrs: - # TODO(@Ericson2314): Make this more modular, and not O(n^2). let + fixedOutputDrv = attrs ? outputHash; + noNonNativeDeps = builtins.length (depsBuildTarget ++ depsBuildTargetPropagated + ++ depsHostHost ++ depsHostHostPropagated + ++ buildInputs ++ propagatedBuildInputs + ++ depsTargetTarget ++ depsTargetTargetPropagated) == 0; + runtimeSensativeIfFixedOutput = fixedOutputDrv -> !noNonNativeDeps; supportedHardeningFlags = [ "fortify" "stackprotector" "pie" "pic" "strictoverflow" "format" "relro" "bindnow" ]; defaultHardeningFlags = lib.remove "pie" supportedHardeningFlags; enabledHardeningOptions = @@ -168,7 +173,11 @@ rec { # A hack to make `nix-env -qa` and `nix search` ignore broken packages. # TODO(@oxij): remove this assert when something like NixOS/nix#1771 gets merged into nix. name = assert validity.handled; name + lib.optionalString - (stdenv.hostPlatform != stdenv.buildPlatform) + # Fixed-output derivations like source tarballs shouldn't get a host + # suffix. But we have some weird ones with run-time deps that are + # just used for their side-affects. Those might as well since the + # hash can't be the same. See #32986. + (stdenv.hostPlatform != stdenv.buildPlatform && runtimeSensativeIfFixedOutput) ("-" + stdenv.hostPlatform.config); builder = attrs.realBuilder or stdenv.shell; |