-
Notifications
You must be signed in to change notification settings - Fork 13k
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 alternative main feature #39337
Conversation
r? @nrc (rust_highfive has picked a reviewer for you, use r? to override) |
c4d03a7
to
08c7384
Compare
@cramertj hmm the only problem I see on travis is that you have exceeded the 100 character line limit, which should be relatively easy to fix. |
Weird. Must be something messed up locally. Thanks for the heads-up! |
08c7384
to
b9ce642
Compare
b9ce642
to
d5da4a5
Compare
@nikomatsakis The issues I mentioned are showing up in travis now that I fixed the line length. |
@@ -468,7 +468,6 @@ pub const BUILTIN_ATTRIBUTES: &'static [(&'static str, AttributeType, AttributeG | |||
|
|||
("cfg", Normal, Ungated), | |||
("cfg_attr", Normal, Ungated), | |||
("main", Normal, Ungated), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems weird that this was ungated. Maybe that has to do with the issue.
Edit: Doesn't look like it. Adding that line back in didn't make any difference.
Fix #29634.
WIP because I'm still getting two errors:
compile-fail/test-warns-dead-code
is giving an unused attribute errorcompile-fail/issue-12997-2
isn't giving the expected mismatched type errorI'm not sure I understand how either of these are related to the
main
feature, so I'm going to return to this tomorrow after I've gotten some sleep. If anyone has an idea might be causing this, I'd appreciate a hint.