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

rust: Update nightly version #204

Merged
merged 1 commit into from
Sep 14, 2022

Conversation

chrysn
Copy link
Member

@chrysn chrysn commented Sep 14, 2022

This allows more precise testing of changes that have stabilized. Concretely, all the changes (cstr in core, generic associated types) that have been stabilized available in the nightly in the form in which they will later be part of the releases.

Testing: CI should do (given it runs through representative builds), but I'm doing some extra tests (building out-of-tree examples) just to ensure that coverage is even better and make it even less likely to cause problems once a first full run is done on murdock.

This allows more precise testing of changes that have stabilized.
@chrysn
Copy link
Member Author

chrysn commented Sep 14, 2022

Whoops, this will probably build-fail because a Rust library hasn't been updated in a while. I'm drilling down on it, probably just needs a RIOT-side round of updates.

False alarm, I was not testing on master but on a branch that left master before the last release.

@chrysn chrysn requested a review from kaspar030 September 14, 2022 16:07
@chrysn
Copy link
Member Author

chrysn commented Sep 14, 2022

My tests wound up positive, and CI is also happy.

Copy link
Contributor

@kaspar030 kaspar030 left a comment

Choose a reason for hiding this comment

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

ACK.

@kaspar030
Copy link
Contributor

bors merge

@bors
Copy link
Contributor

bors bot commented Sep 14, 2022

Build succeeded:

@bors bors bot merged commit ce7a9f2 into RIOT-OS:master Sep 14, 2022
@chrysn chrysn deleted the rust-nightly-20220914 branch September 16, 2022 13:24
chrysn added a commit to chrysn-pull-requests/riotdocker that referenced this pull request Sep 25, 2022
This is what RIOT-OS#204 should have
been, but as that nightly build did not contain that day's rustc, the
intended version was not installed.
bors bot added a commit that referenced this pull request Sep 26, 2022
207: rust: Update nightly version r=kaspar030 a=chrysn

This is what #204 should have been, but as that nightly build did not contain that day's rustc, the intended version was not installed.

---

I did tests back in #204, but they were about "do they existing things still work", not (what I couldn't test easily back then, as it'd have spanned many crates at once) "does it also have the intended effect of allowing stable use of all the new features".

This unblocks RIOT-OS/rust-riot-wrappers#14, and would have made the rollback of RIOT-OS/RIOT#18642 (comment) unnecessary.

Co-authored-by: chrysn <chrysn@fsfe.org>
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.

2 participants