Skip to content

Run build test for x86_64-unknown-redox #1934

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 5 commits into from
Oct 19, 2020
Merged

Conversation

JohnTitor
Copy link
Member

@JohnTitor JohnTitor commented Oct 17, 2020

Fix #1457

@rust-highfive
Copy link

@JohnTitor: no appropriate reviewer found, use r? to override

@JohnTitor
Copy link
Member Author

@bors try

@bors
Copy link
Contributor

bors commented Oct 17, 2020

⌛ Trying commit 999acbe with merge 6c21ab0...

bors added a commit that referenced this pull request Oct 17, 2020
Add `x86_64-unknown-redox` to tier 2 CI
@bors
Copy link
Contributor

bors commented Oct 17, 2020

💔 Test failed - checks-actions

@JohnTitor
Copy link
Member Author

@bors try

bors added a commit that referenced this pull request Oct 17, 2020
Add `x86_64-unknown-redox` to tier 2 CI
@bors
Copy link
Contributor

bors commented Oct 17, 2020

⌛ Trying commit 9bfe05d with merge ff10f1c...

@bors
Copy link
Contributor

bors commented Oct 17, 2020

💔 Test failed - checks-actions

@JohnTitor
Copy link
Member Author

@bors try

@bors
Copy link
Contributor

bors commented Oct 17, 2020

⌛ Trying commit 09d80ab with merge f2e6133...

bors added a commit that referenced this pull request Oct 17, 2020
Add `x86_64-unknown-redox` to tier 2 CI
@bors
Copy link
Contributor

bors commented Oct 17, 2020

💔 Test failed - checks-actions

@JohnTitor
Copy link
Member Author

@bors try

@bors
Copy link
Contributor

bors commented Oct 17, 2020

⌛ Trying commit 8e2b2e1 with merge 069eb9c...

bors added a commit that referenced this pull request Oct 17, 2020
Add `x86_64-unknown-redox` to tier 2 CI
@bors
Copy link
Contributor

bors commented Oct 17, 2020

💔 Test failed - checks-actions

@JohnTitor
Copy link
Member Author

@bors try

@bors
Copy link
Contributor

bors commented Oct 17, 2020

⌛ Trying commit 3230c80 with merge e39184d...

bors added a commit that referenced this pull request Oct 17, 2020
Add `x86_64-unknown-redox` to tier 2 CI
@bors
Copy link
Contributor

bors commented Oct 17, 2020

💔 Test failed - checks-actions

@JohnTitor
Copy link
Member Author

@bors try

@bors
Copy link
Contributor

bors commented Oct 17, 2020

⌛ Trying commit a78203c with merge 90859e2...

bors added a commit that referenced this pull request Oct 17, 2020
Add `x86_64-unknown-redox` to tier 2 CI
@bors
Copy link
Contributor

bors commented Oct 17, 2020

💔 Test failed - checks-actions

@JohnTitor
Copy link
Member Author

@bors try

@bors
Copy link
Contributor

bors commented Oct 17, 2020

⌛ Trying commit 2c4414b with merge 425986d...

bors added a commit that referenced this pull request Oct 17, 2020
Add `x86_64-unknown-redox` to tier 2 CI
@bors
Copy link
Contributor

bors commented Oct 17, 2020

💔 Test failed - checks-actions

@JohnTitor
Copy link
Member Author

Okay, we've found that it seems some items in src/unix/mod.rs aren't defined in redox actually.
I'd downgrade CI to build testing to avoid breakage for now but we should address at some point.

@JohnTitor JohnTitor changed the title Add x86_64-unknown-redox to tier 2 CI Run build test for x86_64-unknown-redox Oct 17, 2020
@JohnTitor
Copy link
Member Author

@bors r+

@bors
Copy link
Contributor

bors commented Oct 17, 2020

📌 Commit 6f1b951 has been approved by JohnTitor

@JohnTitor JohnTitor mentioned this pull request Oct 17, 2020
bors added a commit that referenced this pull request Oct 17, 2020
Run build test for `x86_64-unknown-redox`

Fix #1457
@bors
Copy link
Contributor

bors commented Oct 17, 2020

⌛ Testing commit 6f1b951 with merge fd7e3c3...

@bors
Copy link
Contributor

bors commented Oct 17, 2020

💔 Test failed - checks-cirrus-freebsd-10

@pietroalbini
Copy link
Member

@bors r-

Bors mistakenly thought this didn't fail after a synchronize.

@bors
Copy link
Contributor

bors commented Oct 19, 2020

@pietroalbini: 🔑 Insufficient privileges: Not in reviewers

@JohnTitor
Copy link
Member Author

@bors r-

@JohnTitor
Copy link
Member Author

@bors r+

@bors
Copy link
Contributor

bors commented Oct 19, 2020

📌 Commit 6f1b951 has been approved by JohnTitor

@bors
Copy link
Contributor

bors commented Oct 19, 2020

⌛ Testing commit 6f1b951 with merge 35ee6ed...

@bors
Copy link
Contributor

bors commented Oct 19, 2020

☀️ Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing 35ee6ed to master...

@bors bors merged commit 35ee6ed into rust-lang:master Oct 19, 2020
@JohnTitor JohnTitor deleted the redox branch October 19, 2020 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Temporarily disable redox target (tracking issue)
4 participants