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 alternative main feature #39337

Closed
wants to merge 1 commit into from

Conversation

cramertj
Copy link
Member

Fix #29634.

WIP because I'm still getting two errors:
compile-fail/test-warns-dead-code is giving an unused attribute error
compile-fail/issue-12997-2 isn't giving the expected mismatched type error

I'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.

@rust-highfive
Copy link
Collaborator

r? @nrc

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

@cramertj cramertj changed the title Remove alternative main feature [WIP] Remove alternative main feature Jan 27, 2017
@cramertj cramertj force-pushed the cramertj/no-more-main branch from c4d03a7 to 08c7384 Compare January 27, 2017 08:50
@nikomatsakis
Copy link
Contributor

r? @nikomatsakis

@rust-highfive rust-highfive assigned nikomatsakis and unassigned nrc Jan 27, 2017
@nikomatsakis
Copy link
Contributor

@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.

@cramertj
Copy link
Member Author

Weird. Must be something messed up locally. Thanks for the heads-up!

@cramertj cramertj force-pushed the cramertj/no-more-main branch from 08c7384 to b9ce642 Compare January 27, 2017 16:26
@cramertj cramertj force-pushed the cramertj/no-more-main branch from b9ce642 to d5da4a5 Compare January 27, 2017 17:02
@cramertj cramertj changed the title [WIP] Remove alternative main feature Remove alternative main feature Jan 27, 2017
@cramertj
Copy link
Member Author

@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),
Copy link
Member Author

@cramertj cramertj Jan 27, 2017

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.

@cramertj
Copy link
Member Author

So, it turns out there's a bunch of stuff internal to the compiler that uses this attribute, so I'm going to close this for now while I look through all of it and figure out what to do. Examples: here, here.

@cramertj cramertj closed this Jan 28, 2017
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.

4 participants