swendel/nixpkgs
1
0
Fork 0
mirror of https://github.com/SebastianWendel/nixpkgs.git synced 2024-10-01 09:30:20 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #33589 from dtzWill/fix/fixed-output-cross-name

Browse source 
make-derivation: Don't add host-suffix to fixed-output derivations names
This commit is contained in:
John Ericson 2018-07-09 12:00:50 -04:00 committed by GitHub
parent 58021b77a0 218d4dc154
commit cf72af3d99
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 13 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
pkgs/stdenv/generic/make-derivation.nix
View file

@ -77,8 +77,13 @@ rec {
, ... } @ attrs: , ... } @ attrs:
# TODO(@Ericson2314): Make this more modular, and not O(n^2).
let 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" ]; supportedHardeningFlags = [ "fortify" "stackprotector" "pie" "pic" "strictoverflow" "format" "relro" "bindnow" ];
defaultHardeningFlags = lib.remove "pie" supportedHardeningFlags; defaultHardeningFlags = lib.remove "pie" supportedHardeningFlags;
enabledHardeningOptions = enabledHardeningOptions =
@ -168,7 +173,11 @@ rec {
# A hack to make `nix-env -qa` and `nix search` ignore broken packages. # 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. # TODO(@oxij): remove this assert when something like NixOS/nix#1771 gets merged into nix.
name = assert validity.handled; name + lib.optionalString 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); ("-" + stdenv.hostPlatform.config);
builder = attrs.realBuilder or stdenv.shell; builder = attrs.realBuilder or stdenv.shell;

9
pkgs/top-level/splice.nix
View file

@ -31,13 +31,8 @@ let
then defaultBuildHostScope then defaultBuildHostScope
else assert pkgs.hostPlatform == pkgs.buildPlatform; defaultHostTargetScope; else assert pkgs.hostPlatform == pkgs.buildPlatform; defaultHostTargetScope;
defaultHostHostScope = {}; # unimplemented defaultHostHostScope = {}; # unimplemented
# TODO(@Ericson2314): we shouldn't preclude run-time fetching by removing defaultHostTargetScope = pkgs // pkgs.xorg;
# these attributes. We should have a more general solution for selecting defaultTargetTargetScope = pkgs.targetPackages // pkgs.targetPackages.xorg or {};
# whether `nativeDrv` or `crossDrv` is the default in `defaultScope`.
pkgsWithoutFetchers = lib.filterAttrs (n: _: !lib.hasPrefix "fetch" n) pkgs;
targetPkgsWithoutFetchers = lib.filterAttrs (n: _: !lib.hasPrefix "fetch" n) pkgs.targetPackages;
defaultHostTargetScope = pkgsWithoutFetchers // pkgs.xorg;
defaultTargetTargetScope = targetPkgsWithoutFetchers // targetPkgsWithoutFetchers.xorg or {};
splicer = pkgsBuildBuild: pkgsBuildHost: pkgsBuildTarget: splicer = pkgsBuildBuild: pkgsBuildHost: pkgsBuildTarget:
pkgsHostHost: pkgsHostTarget: pkgsHostHost: pkgsHostTarget: