-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
Update Clippy and cargo #67661
Update Clippy and cargo #67661
Conversation
|
@bors r+ p=1 |
📌 Commit cc454f841f1d76324ab37c7e40fec9c2eb358fe8 has been approved by |
⌛ Testing commit cc454f841f1d76324ab37c7e40fec9c2eb358fe8 with merge f473bfc689dbd35766ab9394a5a71b03e72003c5... |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
💔 Test failed - checks-azure |
cc454f8
to
14084c9
Compare
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
Ugh, might be related to latest rustup? |
The ICE on the test is only shown on rustc CI (the situation is same as #64963 (comment)). |
backtrace
|
You may be able fix it by erasing regions on The reason it fails on CI and not in clippy is probably because that is a |
Thanks! But the suggested fix didn't work well (I tested dirty-submodule with - let key_type = substs.type_at(0);
+ let key_type = cx.tcx.erase_regions(&substs.type_at(0)); any thoughts? |
Well... as far as I can tell the clippy code is actually not really interested in the frozenness of the reference, but on the frozenness of the things behind it. Maybe if https://github.com/rust-lang/rust-clippy/blame/0fcb5304e27a747267276cdbac720f4bd2672172/clippy_lints/src/mut_key.rs#L114 returned an |
yeah, it seems an ICE has been gone, at least on my local. I submitted a fix rust-lang/rust-clippy#4964 |
Possibly fix an ICE on test Fix a potential ICE on test with debug assertion, caused on rust-lang/rust#67661. r? @oli-obk changelog: none
14084c9
to
107106a
Compare
It seems the test has been passed (checked on |
@bors r+ p=1 |
📌 Commit 107106a6a3febee0746597ee08b18346d9c46cc9 has been approved by |
⌛ Testing commit 107106a6a3febee0746597ee08b18346d9c46cc9 with merge 9f05033d2755d9a938a74eb696e883c9caf83a52... |
It seems rustup for latest master is needed... (travis on clippy) |
107106a
to
f541e91
Compare
feel free to r- if someone has a concern |
📌 Commit f541e91 has been approved by |
🌲 The tree is currently closed for pull requests below priority 10, this pull request will be tested once the tree is reopened |
Homu queue points a resolved comment so it's unintended? |
Update Clippy and cargo includes latest rustup and resolves dependency conflicts with cargo Closes #67541 r? @oli-obk @Manishearth
☀️ Test successful - checks-azure |
📣 Toolstate changed by #67661! Tested on commit 25434f8. 🎉 clippy-driver on windows: build-fail → test-pass (cc @mcarton @oli-obk @Manishearth @flip1995 @yaahc @phansch @llogiq, @rust-lang/infra). |
Tested on commit rust-lang/rust@25434f8. Direct link to PR: <rust-lang/rust#67661> 🎉 clippy-driver on windows: build-fail → test-pass (cc @mcarton @oli-obk @Manishearth @flip1995 @yaahc @phansch @llogiq, @rust-lang/infra). 🎉 clippy-driver on linux: build-fail → test-pass (cc @mcarton @oli-obk @Manishearth @flip1995 @yaahc @phansch @llogiq, @rust-lang/infra). 💔 rls on windows: test-pass → build-fail (cc @Xanewok, @rust-lang/infra). 💔 rls on linux: test-pass → build-fail (cc @Xanewok, @rust-lang/infra).
includes latest rustup and resolves dependency conflicts with cargo
Closes #67541
r? @oli-obk @Manishearth