-
Notifications
You must be signed in to change notification settings - Fork 23
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
Support RUSTC_WRAPPER #26
Comments
With rust-lang/cargo#9601, it should be enough to just check RUSTC_WRAPPER -- cargo will set it to whatever wrapper value it ended up computing based on the config files. |
Turns out that by not supporting RUSTC_WRAPPER autocfg has accidentally worked around rust-lang/rust-analyzer#12973. |
The lack of support for RUSTC_WRAPPER is unfortunately preventing us from removing some hacks from the Miri driver: rust-lang/miri#3411. |
Ok, I'll work on this -- shouldn't be too hard, I hope... |
I think it should be fairly simple, yeah. Here's how anyhow does it. |
Thanks a lot!
Now I will have to check what this RUSTC_WORKSPACE_WRAPPER thing is, I never heard about this before...
|
The summary in rust-lang/cargo#8143 is pretty good. I don't know if anyone would really use that in a way that affects |
Oh, so this doesn't have anything to do with workspaces, it also works for single-crate projects that don't use workspaces. That 's a very confusing name. Thanks for the link! |
In rust-num/num-complex#83, they reported problems detecting features under
cargo no-std-check
, which I found to be usingRUSTC_WRAPPER
to rewrite arguments for a fake target. This seems like something thatautocfg
could reasonably support. When present and non-empty, we would just wrap commands as$RUSTC_WRAPPER $RUSTC <args...>
.An alternate environment name is
CARGO_BUILD_RUSTC_WRAPPER
, or it can be set in.cargo/config
-- but we don't have any support to find or parse config files yet.The text was updated successfully, but these errors were encountered: