New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
clang-stdenv: move libstdcxxHook to cc-wrapper #29246
Conversation
|
||
libcxxClang = ccWrapperFun { | ||
cc = self.clang-unwrapped; | ||
isClang = true; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this because it figures it out automatically?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, it's not necessary.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
I bisected a problem I'm having in one of my local environments to this commit.
I only just started looking into it, but I thought I'd check if anyone had any thoughts. I'm creating an environment with gcc for building, and clang for tooling. |
Motivation for this change
Fixes #29204
Things done
build-use-sandbox
innix.conf
on non-NixOS)tests.cc-wrapper
)nix-shell -p nox --run "nox-review wip"
./result/bin/
)