From 77b6f6734f830058b2375cfc96b5e0fb482cc1c2 Mon Sep 17 00:00:00 2001 From: Jade Lovelace Date: Tue, 15 Oct 2024 15:45:42 -0700 Subject: [PATCH] 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 --- flake.nix | 4 ++++ package.nix | 6 +++++- 2 files changed, 9 insertions(+), 1 deletion(-) 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 ];