Skip to content
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

Enable rustc internal lints #3966

Merged
merged 4 commits into from
Apr 17, 2019
Merged

Enable rustc internal lints #3966

merged 4 commits into from
Apr 17, 2019

Conversation

flip1995
Copy link
Member

Closes #3965

I'm not 100% sure if enabling the -Zunstable-options flag in the .cargo/config file is the right place.

@flip1995 flip1995 added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Apr 15, 2019
@oli-obk
Copy link
Contributor

oli-obk commented Apr 15, 2019

I'm not 100% sure if enabling the -Zunstable-options flag in the .cargo/config file is the right place.

It seems not :D the stable compiler test doesn't work

@flip1995
Copy link
Member Author

Any idea how we can hack in the -Zunstable-options so it is only set in clippy_lints? Adding a clippy_lints/.cargo/config file doesn't work, since cargo is run from the root of the project and therefore ignores the config file in clippy_lints.

Also adding a clippy_dev/.cargo/config with

[build]
rustflags = []

does not work, since the config file in the root gets unified with this file.

Adding it to the build script also seems not really possible:

rustc-flags=FLAGS is a set of flags passed to the compiler, only -l and -L flags are supported.
https://doc.rust-lang.org/cargo/reference/build-scripts.html#outputs-of-the-build-script

@oli-obk
Copy link
Contributor

oli-obk commented Apr 16, 2019

Maybe clippy_dev should not be using stable. I mean there's no need to download a full stable compiler in CI just for that crate.

@flip1995
Copy link
Member Author

It was just changed to stable recently: #3885

I think removing the rust-toolchain file again would be the easiest fix.

@oli-obk
Copy link
Contributor

oli-obk commented Apr 16, 2019

sgtm

@oli-obk
Copy link
Contributor

oli-obk commented Apr 16, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Apr 16, 2019

📌 Commit 118f7d5 has been approved by oli-obk

@bors
Copy link
Contributor

bors commented Apr 16, 2019

⌛ Testing commit 118f7d5 with merge 29893fe...

bors added a commit that referenced this pull request Apr 16, 2019
Enable rustc internal lints

Closes #3965

I'm not 100% sure if enabling the `-Zunstable-options` flag in the `.cargo/config` file is the right place.
@bors
Copy link
Contributor

bors commented Apr 16, 2019

💥 Test timed out

@phansch
Copy link
Member

phansch commented Apr 17, 2019

@bors retry

@bors
Copy link
Contributor

bors commented Apr 17, 2019

⌛ Testing commit 118f7d5 with merge 27d62cf...

bors added a commit that referenced this pull request Apr 17, 2019
Enable rustc internal lints

Closes #3965

I'm not 100% sure if enabling the `-Zunstable-options` flag in the `.cargo/config` file is the right place.
@bors
Copy link
Contributor

bors commented Apr 17, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: oli-obk
Pushing 27d62cf to master...

@bors bors merged commit 118f7d5 into rust-lang:master Apr 17, 2019
This was referenced Apr 17, 2019
@flip1995 flip1995 deleted the internal_lints branch April 17, 2019 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use rustc internal lints in Clippy
4 participants