Skip to content

Fix an unresolved import issue with enabled use_extern_macros #50355

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

Merged
merged 2 commits into from
May 2, 2018

Conversation

petrochenkov
Copy link
Contributor

@petrochenkov petrochenkov commented May 1, 2018

This is a kinda ugly special-purpose solution that will break if we suddenly add a fourth namespace, but I hope to come up with something more general if I get to import resolution refactoring this summer.

Fixes #50187 thus removing a blocker for stabilization of use_extern_macros

@petrochenkov
Copy link
Contributor Author

r? @oli-obk maybe

@petrochenkov petrochenkov added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 1, 2018
@oli-obk
Copy link
Contributor

oli-obk commented May 1, 2018

@bors r+

@bors
Copy link
Collaborator

bors commented May 1, 2018

📌 Commit d5e3115 has been approved by oli-obk

@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 May 1, 2018
@bors
Copy link
Collaborator

bors commented May 2, 2018

⌛ Testing commit d5e3115 with merge 8a37c75...

bors added a commit that referenced this pull request May 2, 2018
Fix an unresolved import issue with enabled `use_extern_macros`

This is a kinda ugly special-purpose solution that will break if we suddenly add a fourth namespace, but I hope to come up with something more general if I get to import resolution refactoring this summer.

Fixes #50187 thus removing a blocker for stabilization of `use_extern_macros`
@bors
Copy link
Collaborator

bors commented May 2, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: oli-obk
Pushing 8a37c75 to master...

@bors bors merged commit d5e3115 into rust-lang:master May 2, 2018
@petrochenkov petrochenkov deleted the 50187 branch June 5, 2019 16:05
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.

3 participants