aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJade Lovelace <lix@jade.fyi>2024-10-15 15:45:42 -0700
committerJade Lovelace <lix@jade.fyi>2024-10-18 15:40:22 -0700
commit77b6f6734f830058b2375cfc96b5e0fb482cc1c2 (patch)
treee9d701278360e392cc75b10233608b7aa9bfc896
parente2d00ac3a805bef7af8e7ec765af0d8ba34f4b3c (diff)
packaging: remove workaround for clang stdenv asserts
This workaround was for the stdenv not being set when callPackage'ing package.nix for some of the stranger CI outputs. Change-Id: I2acdd6efa721b90dd3cb04358544d25d591ff084
-rw-r--r--flake.nix4
-rw-r--r--package.nix6
2 files changed, 9 insertions, 1 deletions
diff --git a/flake.nix b/flake.nix
index f77f848ca..9fe752dc8 100644
--- a/flake.nix
+++ b/flake.nix
@@ -269,6 +269,8 @@
nix = pkgs.callPackage ./package.nix {
inherit versionSuffix officialRelease buildUnreleasedNotes;
inherit (pkgs) build-release-notes;
+ # Required since we don't support gcc stdenv
+ stdenv = pkgs.clangStdenv;
internalApiDocs = true;
busybox-sandbox-shell = pkgs.busybox-sandbox-shell;
};
@@ -326,6 +328,8 @@
inherit (nixpkgs) pkgs;
in
pkgs.callPackage ./package.nix {
+ # Required since we don't support gcc stdenv
+ stdenv = pkgs.clangStdenv;
versionSuffix = "";
lintInsteadOfBuild = true;
};
diff --git a/package.nix b/package.nix
index e869fb74f..4bce7cb5e 100644
--- a/package.nix
+++ b/package.nix
@@ -112,7 +112,7 @@
}:
# gcc miscompiles coroutines at least until 13.2, possibly longer
-assert stdenv.cc.isClang || lintInsteadOfBuild || internalApiDocs;
+assert stdenv.cc.isClang;
let
inherit (__forDefaults) canRunInstalled;
@@ -274,6 +274,10 @@ stdenv.mkDerivation (finalAttrs: {
++ lib.optionals lintInsteadOfBuild [
# required for a wrapped clang-tidy
llvmPackages.clang-tools
+ # load-bearing order (just as below); the actual stdenv wrapped clang
+ # needs to precede the unwrapped clang in PATH such that calling `clang`
+ # can compile things.
+ stdenv.cc
# required for run-clang-tidy
llvmPackages.clang-unwrapped
];