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

Add x86_64-unknown-redox to build manifest target list #43950

Merged
merged 1 commit into from
Aug 20, 2017

Conversation

jackpot51
Copy link
Contributor

This should fix the issue of x86_64-unknown-redox not being available for rustup target add

@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

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

@nikomatsakis
Copy link
Contributor

r? @alexcrichton -- I know nothing of this file, though this looks "obviously right"

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Aug 17, 2017

📌 Commit 2fdade4 has been approved by alexcrichton

@jackpot51
Copy link
Contributor Author

Thanks for being quick @nikomatsakis @alexcrichton. I look forward to having it tomorrow!

😄

@frewsxcv
Copy link
Member

@bors rollup

aidanhs added a commit to aidanhs/rust that referenced this pull request Aug 18, 2017
Add x86_64-unknown-redox to build manifest target list

This should fix the issue of `x86_64-unknown-redox` not being available for `rustup target add`
bors added a commit that referenced this pull request Aug 18, 2017
Rollup of 4 pull requests

- Successful merges: #43919, #43936, #43950, #43978
- Failed merges:
@bors
Copy link
Contributor

bors commented Aug 19, 2017

⌛ Testing commit 2fdade4 with merge b963a5930799f81a2a37865372253e75ceb3acda...

@bors
Copy link
Contributor

bors commented Aug 19, 2017

💔 Test failed - status-appveyor

@kennytm
Copy link
Member

kennytm commented Aug 19, 2017

@bors retry #43985

@bors
Copy link
Contributor

bors commented Aug 20, 2017

⌛ Testing commit 2fdade4 with merge e822e4e...

bors added a commit that referenced this pull request Aug 20, 2017
Add x86_64-unknown-redox to build manifest target list

This should fix the issue of `x86_64-unknown-redox` not being available for `rustup target add`
@bors
Copy link
Contributor

bors commented Aug 20, 2017

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

@bors bors merged commit 2fdade4 into rust-lang:master Aug 20, 2017
@jackpot51 jackpot51 deleted the redox_docker branch September 12, 2017 01:45
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.

7 participants