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

Update from Rust 1.43 to 1.46 and improve packaging in minor ways #12

Merged
merged 3 commits into from
Oct 13, 2020

Conversation

thombles
Copy link
Member

In particular, toolchains are now installed against their specific version number so they can coexist. This is particularly useful in CI environments where different branches may be on different versions.

The good news: The rust hacks were upstreamed so we no longer need to patch rustc, nor do we need any -Z flags. Since the -Z flag is no longer needed, we are now building the stable channel.

The bad news: There is no "good" LLVM commit to target due to a bug. Here we are targeting the 5.3-RELEASE tag, which should match Xcode 12 pretty closely, and using a patch to allow the rust compilation to proceed. See the comments in the patch for a full explanation.

I have confirmed that Rust code built with this toolchain can be included in bitcode apps, deployed to physical hardware, and rebuilt from bitcode.

In particular, toolchains are now installed against their specific
version number, so they can coexist. This is particularly useful in CI
environments where different branches may be on different versions.
Copy link
Member

@hamchapman hamchapman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple of documentation/comment nits but LGTM!

Nice job getting to something that works 🙌

config.sh Outdated Show resolved Hide resolved
dist.sh Outdated Show resolved Hide resolved
Copy link

@konstantinbe konstantinbe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, LGTM. 👍

Copy link
Member

@connorpower connorpower left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@thombles thombles merged commit 50b0403 into master Oct 13, 2020
@thombles thombles deleted the tk/1.46 branch October 13, 2020 23:03
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.

4 participants