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

Decides main name by target #5691

Closed
wants to merge 3 commits into from
Closed

Conversation

ILyoan
Copy link
Contributor

@ILyoan ILyoan commented Apr 3, 2013

Existing rust code decides main name by host environment of rustc.
I think it should be chosen by build target.

This patch is also removing one of the android hacks that is not necessary any longer(I think it was not necessary from the first).

@ILyoan
Copy link
Contributor Author

ILyoan commented Apr 4, 2013

Add a commit that fixes a test fail.
r?

bors added a commit that referenced this pull request Apr 4, 2013
Existing rust code decides main name by host environment of rustc.
I think it should be chosen by build target.

This patch is also removing one of the android hacks that is not necessary any longer(I think it was not necessary from the first).
@bors bors closed this Apr 4, 2013
tesuji pushed a commit to tesuji/rustc that referenced this pull request Jun 9, 2020
peter-lyons-kehl added a commit to coop-rs/rust that referenced this pull request Feb 21, 2023
peter-lyons-kehl added a commit to coop-rs/rust that referenced this pull request Mar 22, 2023
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.

3 participants