diff options
author | Adam Joseph <adam@westernsemico.com> | 2022-04-10 01:59:20 -0700 |
---|---|---|
committer | sternenseemann <sternenseemann@systemli.org> | 2022-05-25 09:45:42 +0200 |
commit | c0085404bd9dd6721eed9b6a5a36a310ca31b286 (patch) | |
tree | 8084a681e6887903cc3206bf581c7bcc0b0bf153 /lib/systems | |
parent | f3fdc04f112f73ecc3a84b147837cbaa680b8fdb (diff) |
lib/systems/inspect.nix: remove isPowerPC
Very confusingly, the `isPowerPC` predicate in `lib/systems/inspect.nix` does *not* match `powerpc64le`! This is because `isPowerPC` is defined as isPowerPC = { cpu = cpuTypes.powerpc; }; Where `cpuTypes.powerpc` is: { bits = 32; significantByte = bigEndian; family = "power"; }; This means that the `isPowerPC` predicate actually only matches the subset of machines marketed under this name which happen to be 32-bit and running in big-endian mode which is equivalent to: with stdenv.hostPlatform; isPower && isBigEndian && is32bit This seems like a sharp edge that people could easily cut themselves on. In fact, that has already happened: in `linux/kernel/common-config.nix` there is a test which will always fail: (stdenv.hostPlatform.isPowerPC && stdenv.hostPlatform.is64bit) A more subtle case of the strict isPowerPC being used instead of the moreg general isPower accidentally are the GHC expressions: Update pkgs/development/compilers/ghc/8.10.7.nix Update pkgs/development/compilers/ghc/8.8.4.nix Update pkgs/development/compilers/ghc/9.2.2.nix Update pkgs/development/compilers/ghc/9.0.2.nix Update pkgs/development/compilers/ghc/head.nix Since the remaining legitimate use sites of isPowerPC are so few, remove the isPowerPC predicate completely. The alternative expression above is noted in the release notes as an alternative. Co-authored-by: sternenseemann <sternenseemann@systemli.org>
Diffstat (limited to 'lib/systems')
-rw-r--r-- | lib/systems/inspect.nix | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/lib/systems/inspect.nix b/lib/systems/inspect.nix index 00cbe4f012cba..e5bd879e2c42a 100644 --- a/lib/systems/inspect.nix +++ b/lib/systems/inspect.nix @@ -11,7 +11,6 @@ rec { isi686 = { cpu = cpuTypes.i686; }; isx86_32 = { cpu = { family = "x86"; bits = 32; }; }; isx86_64 = { cpu = { family = "x86"; bits = 64; }; }; - isPowerPC = { cpu = cpuTypes.powerpc; }; isPower = { cpu = { family = "power"; }; }; isPower64 = { cpu = { family = "power"; bits = 64; }; }; isx86 = { cpu = { family = "x86"; }; }; |