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

Remove newly stabilized features #967

Closed
wants to merge 2 commits into from
Closed

Remove newly stabilized features #967

wants to merge 2 commits into from

Conversation

jhpratt
Copy link
Contributor

@jhpratt jhpratt commented Apr 9, 2019

transpose_result and try_from have both been stabilized in 1.34. This removes the compiler flags.

Related: Compile on stable Rust (#19).

@jebrosen jebrosen self-assigned this Apr 10, 2019
@jebrosen jebrosen modified the milestone: 0.5.0 Apr 10, 2019
@jebrosen
Copy link
Collaborator

jebrosen commented Apr 10, 2019

Merged in 13d75ba.

... and I forgot that this affects the minimum supported rustc 🙄.

@jebrosen jebrosen added the pr: merged This pull request was merged manually. label Apr 10, 2019
@jhpratt
Copy link
Contributor Author

jhpratt commented Apr 10, 2019

@jebrosen Rollback? The PR is still open and can be merged at a later date (likely around the time 0.5 comes out).

@jebrosen
Copy link
Collaborator

If you would change the minimum nightly in core/lib/build.rs that would be awesome, and I'll merge it again later. It looks like rustc 1.34.0-nightly (00aae71f5 2019-02-25) is sufficient.

@jebrosen jebrosen removed the pr: merged This pull request was merged manually. label Apr 10, 2019
@jhpratt
Copy link
Contributor Author

jhpratt commented Apr 10, 2019

Bumped.

@@ -1,6 +1,5 @@
#![feature(proc_macro_diagnostic, proc_macro_span)]
#![feature(crate_visibility_modifier)]
#![feature(transpose_result)]
Copy link
Collaborator

Choose a reason for hiding this comment

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

This bumps the minimum rustc for codegen, but only to around 2019-01-13. Since there are a few other PRs open that affect the minimum rustc, we can just take care of this when merging to avoid conflicts.

@jebrosen
Copy link
Collaborator

Merged in 9b219dd.

@jebrosen jebrosen closed this Apr 13, 2019
@jebrosen jebrosen added the pr: merged This pull request was merged manually. label Apr 13, 2019
@jhpratt jhpratt deleted the unnecessary-features branch April 16, 2019 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: merged This pull request was merged manually.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants