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 to riot-wrappers 0.7.15 #17514

Merged
merged 1 commit into from
Jan 17, 2022

Conversation

chrysn
Copy link
Member

@chrysn chrysn commented Jan 13, 2022

Contribution description

This is a simple cargo update for a newly released riot-wrappers; the only thing non-simple about is is that cstr_core is still held at 0.2.4 awaiting roll-out of the latest CI builders (does no harm, this is just for anyone curious why cargo update would still indicate a change for them).

Testing procedure

  • Green CI

Issues/PRs references

This unblocks #16458

@github-actions github-actions bot added Area: examples Area: Example Applications Area: tests Area: tests and testing framework labels Jan 13, 2022
@chrysn chrysn added Area: Rust Area: Rust wrapper CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer and removed Area: tests Area: tests and testing framework Area: examples Area: Example Applications labels Jan 13, 2022
@chrysn chrysn requested a review from maribu January 13, 2022 17:56
Copy link
Member

@maribu maribu left a comment

Choose a reason for hiding this comment

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

ACK, thanks for the quick fix!

@benpicco benpicco enabled auto-merge January 13, 2022 18:14
@chrysn
Copy link
Member Author

chrysn commented Jan 14, 2022

Hmpf, fails-to-build-with-old-nightly again. I had test in place this time to prevent that, but if the riot/riotbuild:latest image are out there with the new nightly already (as they've been for some days), there's little I can do other than running all running changes to riot-wrappers through actual murdock.

I could try to publish a version in-between (which is a slight inconvenience, as Rust for some reasons only allows version numbers of up to 3 components), but given how half the workers already manage, and workers still being affected by mysterious errors, I hope this can resolve itself when the remaining workers (the mobi ones) are upgraded.

@chrysn chrysn added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jan 14, 2022
@chrysn
Copy link
Member Author

chrysn commented Jan 15, 2022

This will need all workers on a recent riotdocker. (Just in case s.o can ping the admins, I'm ofl the PC and thus the chat right now).

@chrysn chrysn added State: waiting for CI update State: The PR requires an Update to CI to be performed first CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR State: waiting for CI update State: The PR requires an Update to CI to be performed first labels Jan 15, 2022
@benpicco benpicco merged commit 7cd9b6c into RIOT-OS:master Jan 17, 2022
@chrysn chrysn deleted the riot-wrappers-0.7.15 branch January 17, 2022 10:18
@OlegHahm OlegHahm added this to the Release 2022.04 milestone Apr 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Rust Area: Rust wrapper CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants