Skip to content

Improve the detailed error message for E0432 #29796

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 1 commit into from
Nov 17, 2015

Conversation

barosl
Copy link
Contributor

@barosl barosl commented Nov 12, 2015

The command-line error message for E0432 does mention the possibility of missing the extern crate declaration, but the detailed error message for it doesn't.

Fixes #29517.

The command-line error message for E0432 does mention the possibility of
missing the `extern crate` declaration, but the detailed error message
for it doesn't.

Fixes rust-lang#29517.
@rust-highfive
Copy link
Contributor

r? @alexcrichton

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

@barosl
Copy link
Contributor Author

barosl commented Nov 12, 2015

r? @steveklabnik

@steveklabnik
Copy link
Member

@bors: r+ rollup

@bors
Copy link
Collaborator

bors commented Nov 16, 2015

📌 Commit 0b51064 has been approved by steveklabnik

@bors
Copy link
Collaborator

bors commented Nov 16, 2015

⌛ Testing commit 0b51064 with merge 3f2ecad...

@bors
Copy link
Collaborator

bors commented Nov 16, 2015

💔 Test failed - auto-win-gnu-64-nopt-t

@alexcrichton
Copy link
Member

@bors: retry

On Mon, Nov 16, 2015 at 12:18 PM, bors [email protected] wrote:

[image: 💔] Test failed - auto-win-gnu-64-nopt-t
http://buildbot.rust-lang.org/builders/auto-win-gnu-64-nopt-t/builds/2111


Reply to this email directly or view it on GitHub
#29796 (comment).

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Nov 16, 2015
…te, r=steveklabnik

The command-line error message for E0432 does mention the possibility of missing the `extern crate` declaration, but the detailed error message for it doesn't.

Fixes rust-lang#29517.
bors added a commit that referenced this pull request Nov 16, 2015
@bors bors merged commit 0b51064 into rust-lang:master Nov 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants