diff options
author | Jade Lovelace <lix@jade.fyi> | 2024-10-15 15:45:42 -0700 |
---|---|---|
committer | Jade Lovelace <lix@jade.fyi> | 2024-10-18 15:40:22 -0700 |
commit | 77b6f6734f830058b2375cfc96b5e0fb482cc1c2 (patch) | |
tree | e9d701278360e392cc75b10233608b7aa9bfc896 | |
parent | e2d00ac3a805bef7af8e7ec765af0d8ba34f4b3c (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.nix | 4 | ||||
-rw-r--r-- | package.nix | 6 |
2 files changed, 9 insertions, 1 deletions
@@ -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 ]; |