Skip to content

Fix importing inaccessible extern crates (with a warning) - #37023

Merged
bors merged 2 commits into
rust-lang:masterfrom
jseyfried:fix_extern_crate_back_compat
Oct 13, 2016
Merged

Fix importing inaccessible extern crates (with a warning)#37023
bors merged 2 commits into
rust-lang:masterfrom
jseyfried:fix_extern_crate_back_compat

Conversation

@jseyfried

@jseyfriedjseyfried commented Oct 7, 2016

Copy link
Copy Markdown
Contributor

#35776 broke importing inaccessible extern crates, which should work with a warning (c.f. #31362).

Fixes#36747, fixes#37020, and fixes#37021.
r? @nrc

@nikomatsakisnikomatsakis added beta-nominated Nominated for backporting to the compiler in the beta channel. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Oct 7, 2016
@nikomatsakis

Copy link
Copy Markdown
Contributor

Nominating for backport because regression.

@TimNN

TimNN commented Oct 7, 2016

Copy link
Copy Markdown
Contributor

Fixes #37018 as well.

@TimNNTimNN mentioned this pull request Oct 7, 2016
@TimNN

TimNN commented Oct 7, 2016

Copy link
Copy Markdown
Contributor

And also fixes #37021.

@alexcrichton

Copy link
Copy Markdown
Member

Thanks for the quick fixes @jseyfried!

@nrc

nrc commented Oct 11, 2016

Copy link
Copy Markdown
Member

@bors: r+

@bors

bors commented Oct 11, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit 4080efd has been approved by nrc

@nikomatsakisnikomatsakis added the beta-accepted Accepted for backporting to the compiler in the beta channel. label Oct 11, 2016
@nikomatsakis

Copy link
Copy Markdown
Contributor

Marking as beta-accepted. Short patch, fixes regression.

cc @rust-lang/compiler

sophiajt pushed a commit to sophiajt/rust that referenced this pull request Oct 11, 2016
…pat, r=nrc
Fix importing inaccessible `extern crate`s (with a warning)
Fixesrust-lang#36747, fixesrust-lang#37020, and fixesrust-lang#37021.
r? @nrc
@sophiajtsophiajt mentioned this pull request Oct 11, 2016
bors added a commit that referenced this pull request Oct 12, 2016
Rollup of 10 pull requests
- Successful merges: #36692, #36743, #36762, #36991, #37023, #37050, #37056, #37064, #37066, #37067
- Failed merges:
alexcrichton added a commit to alexcrichton/rust that referenced this pull request Oct 12, 2016
…pat, r=nrc
Fix importing inaccessible `extern crate`s (with a warning)
Fixesrust-lang#36747, fixesrust-lang#37020, and fixesrust-lang#37021.
r? @nrc
bors added a commit that referenced this pull request Oct 12, 2016
@bors
bors merged commit 4080efd into rust-lang:masterOct 13, 2016
@jseyfried
jseyfried deleted the fix_extern_crate_back_compat branch October 16, 2016 09:30
@brsonbrson mentioned this pull request Oct 19, 2016
@alexcrichtonalexcrichton removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Oct 19, 2016
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

beta-acceptedAccepted for backporting to the compiler in the beta channel.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Regression: no method in scope Regression: unresolved import ICE when documenting crate that triggers PR 31362 warning

6 participants

@jseyfried@nikomatsakis@TimNN@alexcrichton@nrc@bors