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 riscv64gc-unknown-freebsd support #2570

Merged
merged 2 commits into from Dec 9, 2021
Merged

Add riscv64gc-unknown-freebsd support #2570

merged 2 commits into from Dec 9, 2021

Conversation

ghost
Copy link

@ghost ghost commented Nov 27, 2021

@rust-highfive
Copy link

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @JohnTitor (or someone else) soon.

Please see the contribution instructions for more information.

@Amanieu
Copy link
Member

Amanieu commented Dec 4, 2021

This is blocked on rust-lang/rust#91284. Once that PR is merged and is available in a nightly, please add a build-check on CI by adding riscv64gc-unknown-freebsd to RUST_LINUX_NO_CORE_TARGETS in ci/build.sh.

@JohnTitor
Copy link
Member

r? @Amanieu as I'm not sure if I'm available at that time.

@rust-highfive rust-highfive assigned Amanieu and unassigned JohnTitor Dec 5, 2021
@ghost
Copy link
Author

ghost commented Dec 8, 2021

Added it to ci/build.sh.

@Amanieu
Copy link
Member

Amanieu commented Dec 9, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Dec 9, 2021

📌 Commit d565303 has been approved by Amanieu

@bors
Copy link
Contributor

bors commented Dec 9, 2021

⌛ Testing commit d565303 with merge 6f661a8...

@bors
Copy link
Contributor

bors commented Dec 9, 2021

☀️ Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing 6f661a8 to master...

@bors bors merged commit 6f661a8 into rust-lang:master Dec 9, 2021
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.

4 participants