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

Fix items resoution from prelude on the nightly Rust #4226

Merged
merged 1 commit into from Aug 5, 2019

Conversation

vlad20012
Copy link
Member

@vlad20012 vlad20012 commented Aug 5, 2019

Recently, a lot of items in nightly stdlib prelude (libstd/prelude/v1.rs) were duplicate with different cfg attribute
Screen Shot 2019-08-07 at 17 12 22
But we don't support cfg attributes or now (see #1191). That leads to multiple results while name resolution of these items and subsequent failure in name resolution because of multi resolve.
These changes add a temporary fix for correct name resolution for items from the prelude

@Undin Undin added the fix Pull requests that fix some bug(s) label Aug 5, 2019
@Undin Undin added this to the v103 milestone Aug 5, 2019
@Undin Undin self-assigned this Aug 5, 2019
Copy link
Member

@Undin Undin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors r+

bors bot added a commit that referenced this pull request Aug 5, 2019
4226: Fix items resoution from prelude on the nightly Rust r=Undin a=vlad20012



Co-authored-by: vlad20012 <beskvlad@gmail.com>
@bors
Copy link
Contributor

bors bot commented Aug 5, 2019

@bors bors bot merged commit 6b9f2a5 into master Aug 5, 2019
@bors bors bot deleted the fix-prelude-items-resolve-on-nightly branch August 5, 2019 20:48
@Undin Undin added this to Test in To test Aug 6, 2019
@lancelote lancelote moved this from Test to Done in To test Aug 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Pull requests that fix some bug(s)
Projects
To test
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants