Skip to content

Commit

Permalink
Merge pull request #110726 from Ericson2314/no-stdenv-dot-platform
Browse files Browse the repository at this point in the history
stdenv: Remove stdenv.platform
  • Loading branch information
Ericson2314 committed Jan 25, 2021
2 parents afebf1c + fa204a9 commit e8d39a9
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 10 deletions.
6 changes: 3 additions & 3 deletions pkgs/stdenv/darwin/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
assert crossSystem == localSystem;

let
inherit (localSystem) system platform;
inherit (localSystem) system;

bootstrapClangVersion = "7.1.0";

Expand Down Expand Up @@ -148,7 +148,7 @@ in rec {
__extraImpureHostDeps = commonImpureHostDeps;

extraAttrs = {
inherit macosVersionMin platform;
inherit macosVersionMin;
};
overrides = self: super: (overrides self super) // {
inherit ccNoLibcxx;
Expand Down Expand Up @@ -522,7 +522,7 @@ in rec {
extraAttrs = {
libc = pkgs.darwin.Libsystem;
shellPackage = pkgs.bash;
inherit macosVersionMin platform bootstrapTools;
inherit macosVersionMin bootstrapTools;
};

allowedRequisites = (with pkgs; [
Expand Down
9 changes: 2 additions & 7 deletions pkgs/stdenv/linux/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@
assert crossSystem == localSystem;

let
inherit (localSystem) system platform;
inherit (localSystem) system;

commonPreHook =
''
Expand Down Expand Up @@ -111,11 +111,6 @@ let
stdenvNoCC = prevStage.ccWrapperStdenv;
};

extraAttrs = {
# Having the proper 'platform' in all the stdenvs allows getting proper
# linuxHeaders for example.
inherit platform;
};
overrides = self: super: (overrides self super) // { fetchurl = thisStdenv.fetchurlBoot; };
};

Expand Down Expand Up @@ -369,7 +364,7 @@ in
# TODO: remove this!
inherit (prevStage) glibc;

inherit platform bootstrapTools;
inherit bootstrapTools;
shellPackage = prevStage.bash;
};

Expand Down

0 comments on commit e8d39a9

Please sign in to comment.