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

removed struct_variant feature as it is not gated anymore #19014

Closed
wants to merge 1 commit into from
Closed

removed struct_variant feature as it is not gated anymore #19014

wants to merge 1 commit into from

Conversation

Kintaro
Copy link
Contributor

@Kintaro Kintaro commented Nov 16, 2014

The struct_variant is not gated anymore. This commit just removes it and the resulting warnings when compiling rust.

@rust-highfive
Copy link
Collaborator

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @nikomatsakis (or someone else) soon.

@sfackler
Copy link
Member

I don't think this will build. The stage0 compiler is still checking for the feature.

@sinistersnare
Copy link
Contributor

maybe a needs-snapshot tag would be nice?

@Kintaro
Copy link
Contributor Author

Kintaro commented Nov 16, 2014

A right, forgot to switch to my repository for the test build. My bad.
Yeah, the stage0 compiler is still checking for it.

@alexcrichton
Copy link
Member

@Kintaro sadly our automation, @bors, doesn't deal well with deleted source repositories, would you mind reopening this PR? Thanks!

@Kintaro
Copy link
Contributor Author

Kintaro commented Nov 19, 2014

Sorry, had some git issues. Will reopen it.Do I just create a new PR or how do I go about this?

@alexcrichton
Copy link
Member

We chatted on IRC, but posting here for posterity as well. Reopening should do the trick!

@alexcrichton
Copy link
Member

As in, a new PR :)

bors added a commit that referenced this pull request Nov 22, 2014
The struct_variant is not gated anymore. This commit just removes it and the resulting warnings when compiling rust. Now compiles with the snapshot from 11/18 (as opposed to PR #19014)
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.

6 participants