Skip to content

Fix argument handling on Redox #38626

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

Merged
merged 1 commit into from
Dec 28, 2016
Merged

Fix argument handling on Redox #38626

merged 1 commit into from
Dec 28, 2016

Conversation

jackpot51
Copy link
Contributor

After switching the start code to be handled in libc, we are no longer passing in slices as arguments into the libstd main function. This means that handling had to be rewritten to match the unix way of doing things.

Additional commits on this branch are going to be merged in this PR: #38577 (comment)

@rust-highfive
Copy link
Contributor

r? @sfackler

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

@jackpot51
Copy link
Contributor Author

Don't merge yet, I will rebase on master now that #38577 is merged

@jackpot51
Copy link
Contributor Author

Rebased, ready for merge.
r? @alexcrichton

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Collaborator

bors commented Dec 27, 2016

📌 Commit 88df0e3 has been approved by alexcrichton

@jackpot51
Copy link
Contributor Author

Thanks @alexcrichton!

@bors
Copy link
Collaborator

bors commented Dec 28, 2016

⌛ Testing commit 88df0e3 with merge 371f4d6...

bors added a commit that referenced this pull request Dec 28, 2016
Fix argument handling on Redox

After switching the start code to be handled in libc, we are no longer passing in slices as arguments into the libstd main function. This means that handling had to be rewritten to match the unix way of doing things.

Additional commits on this branch are going to be merged in this PR: #38577 (comment)
@bors
Copy link
Collaborator

bors commented Dec 28, 2016

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 371f4d6 to master...

@bors bors merged commit 88df0e3 into rust-lang:master Dec 28, 2016
@jackpot51 jackpot51 deleted the args_fix branch December 28, 2016 18:02
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.

5 participants