Skip to content

Conversation

eddyb
Copy link
Member

@eddyb eddyb commented Aug 14, 2018

Fixes #53333 by resolving ::crate_name in TypeNS alone, which was overlooked in #52923 and didn't break tests, since having use crate_name; and a crate_name value in the same scope is rare.

@rust-highfive
Copy link
Contributor

r? @petrochenkov

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 14, 2018
@eddyb
Copy link
Member Author

eddyb commented Aug 14, 2018

@bors p=10 This bug broke toolstate so I assume we want the fix before next nightly.

@petrochenkov
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Aug 14, 2018

📌 Commit 262392c has been approved by petrochenkov

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 14, 2018
@bors
Copy link
Collaborator

bors commented Aug 14, 2018

⌛ Testing commit 262392c with merge d67ba90...

bors added a commit that referenced this pull request Aug 14, 2018
rustc_resolve: crates only exist in the type namespace.

Fixes #53333 by resolving `::crate_name` in `TypeNS` alone, which was overlooked in #52923 and didn't break tests, since having `use crate_name;` and a `crate_name` value in the same scope is rare.
@bors
Copy link
Collaborator

bors commented Aug 14, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: petrochenkov
Pushing d67ba90 to master...

@bors bors merged commit 262392c into rust-lang:master Aug 14, 2018
@rust-highfive
Copy link
Contributor

📣 Toolstate changed by #53335!

Tested on commit d67ba90.
Direct link to PR: #53335

🎉 clippy-driver on windows: build-fail → test-pass.
🎉 clippy-driver on linux: build-fail → test-pass.
🎉 rls on windows: build-fail → test-pass.
🎉 rls on linux: build-fail → test-pass.

rust-highfive added a commit to rust-lang-nursery/rust-toolstate that referenced this pull request Aug 14, 2018
Tested on commit rust-lang/rust@d67ba90.
Direct link to PR: <rust-lang/rust#53335>

🎉 clippy-driver on windows: build-fail → test-pass.
🎉 clippy-driver on linux: build-fail → test-pass.
🎉 rls on windows: build-fail → test-pass.
🎉 rls on linux: build-fail → test-pass.
@eddyb eddyb deleted the issue-53333 branch August 14, 2018 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants