Skip to content

Conversation

Aaron1011
Copy link
Contributor

Some build scripts rely on the RUSTC binary being the actual compiler
(e.g. parsing the output of 'RUSTC --version'). To prevent clippy from
breaking these build scripts, this commit sets RUSTC_WRAPPER instead.
This will cause Cargo to leave RUSTC unchanged, making the use of clippy
transparent to build scripts.

@Aaron1011
Copy link
Contributor Author

I'm unsure why the test passed on linux, but not OSX.

@oli-obk
Copy link
Contributor

oli-obk commented Sep 28, 2017

Just a new nightly that broke clippy...

Some build scripts rely on the RUSTC binary being the actual compiler
(e.g. parsing the output of 'RUSTC --version'). To prevent clippy from
breaking these build scripts, this commit sets RUSTC_WRAPPER instead.
This will cause Cargo to leave RUSTC unchanged, making the use of clippy
transparent to build scripts.
@Aaron1011
Copy link
Contributor Author

Rebased against master.

@oli-obk oli-obk merged commit 13caa00 into rust-lang:master Sep 29, 2017
@oli-obk
Copy link
Contributor

oli-obk commented Sep 29, 2017

Thanks! This has been a longstanding issue. Good that there's finally a solution

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants