-
Notifications
You must be signed in to change notification settings - Fork 892
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 illumos (amd64) support #2432
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fundamentally this looks good. I will temporarily push some commits later today which verify the CI aspects, and then we're probably good to go. Thank you for this work.
Thanks! Happy to work through any issues that come up in that testing -- just let me know! |
Hi there! Just wanted to check in and see if there's anything I should be doing to help out on this one. |
Hi @jclulow No, there's nothing you need to do yet, I've been awfully remiss here. I said I'd push a test commit and then entirely forgot due to real-life things. Your reminder that this PR is here was all I needed. I've now pushed that test commit so let's see what happens. |
Looks like the goal target is not yet in a stable release. We may need to wait a few weeks. |
Is that because the build process here is installing |
To be more concrete, because this succeeds today:
... this will succeed after the next stable release:
Is that right? Thank you for putting up with my ignorance here! |
Correct, |
Hello! Now that illumos bits are available in a stable Rust version (1.47) I am hoping we can move to get this integrated. I have rebased on current master, and confirmed that I can build and run a Let me know what the next steps are! Thanks. |
Have you confirmed that the |
Sorry, I should have been clearer that I ran it and was able to get it to install a toolchain. The stock instructions from
I stuck a copy of
|
Fantastic! |
Thanks for merging! I had a look around the documentation, and it wasn't clear to me what drives the process for getting a new release that would then appear presumably on https://rustup.rs -- is there a schedule, or a trigger of some kind, for the next rustup release? |
It's nothing so formal. I'm working toward deciding what else needs to go in for 1.23, you can see that in the milestone here: https://github.com/rust-lang/rustup/milestone/4 if all that gets done then I'll definitely be releasing :D |
add illumos (amd64) support
With the integration of rust-lang/rust#71272, illumos is now (as I understand it!) a Tier 2 platform (see also rust-lang/compiler-team#279, rust-lang/rustup-components-history#19, etc). We have target and host binaries being built through CI/CD, which you can see at: https://rust-lang.github.io/rustup-components-history/x86_64-unknown-illumos.html
This change adds support illumos systems to
rustup
, and to the installation wrapper script that folks use via the instructions on the site (i.e.,curl | bash
). I believe it also rigs us up to cross build through the CI system, but I wasn't completely sure if I got this part right so any pointers definitely welcome.Testing Notes
cargo test
on my OpenIndiana (an illumos distribution) build system. It ran a lot of tests; they passed, except for a handful that were ignored.rustup-init
directly, and it was able to install the latest nightly toolchain.rustup-init.sh
directly, and it was able to correctly determine that there were norustup-init
binaries available yet -- but at a URL which seemed to make sense!