Skip to content

extern crate changes. #455

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
Nov 1, 2018
Merged

extern crate changes. #455

merged 2 commits into from
Nov 1, 2018

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Oct 17, 2018

Stabilization PR: rust-lang/rust#54403

Possible upcoming changes that will change this:

`extern crate` items will automatically be added to the extern prelude.
https://github.com/rust-lang/rust/pull/54658
Copy link
Contributor

Choose a reason for hiding this comment

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

This is merged now

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The changes are behind the extern_crate_item_prelude feature flag, so there's not much difference. I don't know what the future holds, there are no tracking issues. I tweaked the wording slightly, but it still seems a little awkward to me. Is the distinction between the --extern added to extern prelude and extern crate being placed as an item in the current scope clear enough? Should it emphasize this distinction more?

Copy link
Contributor

@matthewjasper matthewjasper left a comment

Choose a reason for hiding this comment

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

I'm happy to merge this once the conflict is resolved

@ehuss
Copy link
Contributor Author

ehuss commented Nov 1, 2018

I'm happy to merge this once the conflict is resolved

Oops, I'm so used to bors leaving a message I forget to check.

@matthewjasper matthewjasper merged commit 803412c into rust-lang:master Nov 1, 2018
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.

2 participants