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 updates for 2024.04 #20489

Merged

Conversation

chrysn
Copy link
Member

@chrysn chrysn commented Mar 20, 2024

Contribution description

This is one of the semi-regular updates of riot-sys, -wrappers and other dependencies, split up by

$ find -name Cargo.toml -exec cargo update --manifest-path "{}" --package riot-wrappers --package riot-sys ';'
$ find -name Cargo.toml -exec cargo update --manifest-path "{}" ';'

for cleaner per-commit comments.

This applies one issue from riot-sys (RIOT-OS/rust-riot-sys#43), everything else is maintenance-only.

Testing procedure

Automated tests suffice.

After this is merged

... will be a good time for me to publish updated versions to crates.io, so that users of out-of-tree builds get the same experience as users of in-tree builds for the release.

@github-actions github-actions bot added Area: tests Area: tests and testing framework Area: sys Area: System Area: examples Area: Example Applications labels Mar 20, 2024
@chrysn chrysn added Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: run tests If set, CI server will run tests on hardware for the labeled PR Area: Rust Area: Rust wrapper and removed Area: sys Area: System labels Mar 20, 2024
@chrysn chrysn requested a review from Teufelchen1 March 20, 2024 05:14
@chrysn chrysn changed the title Riot updates before 2024.04 Rust updates for 2024.04 Mar 20, 2024
@chrysn chrysn removed the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Mar 20, 2024
@chrysn
Copy link
Member Author

chrysn commented Mar 20, 2024

Putting this on CI hold because there is one update to riot-wrappers I'd like to get in before (RIOT-OS/rust-riot-wrappers#82 saves 800 bytes on minimal builds).

chrysn added 3 commits March 20, 2024 15:59
Changes pulled by this are:
* add hashes to riot-sys headers <RIOT-OS/rust-riot-sys#43>
* administrative changes (riot-sys 0.7.11, CI fixes)
All changes appear to be minor; some crates that were previously part of
the build dependencies twice can now be shared between build
dependencies.
Changes pulled by this are:
* optimizations around panicking <RIOT-OS/rust-riot-wrappers#82>
* administrative changes (CI fixes)
@chrysn chrysn force-pushed the riot-updates-before-2024.04 branch from 2c12002 to a0ee4de Compare March 20, 2024 06:00
@chrysn chrysn added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Mar 20, 2024
@chrysn
Copy link
Member Author

chrysn commented Mar 20, 2024

Ready again, awaiting CI confirmation.

@riot-ci
Copy link

riot-ci commented Mar 20, 2024

Murdock results

✔️ PASSED

a0ee4de rust: Selectively update riot-wrappers

Success Failures Total Runtime
10011 0 10012 10m:47s

Artifacts

@chrysn chrysn added this pull request to the merge queue Mar 20, 2024
Merged via the queue into RIOT-OS:master with commit d8e693c Mar 20, 2024
27 checks passed
@chrysn chrysn deleted the riot-updates-before-2024.04 branch March 20, 2024 22:18
@MrKevinWeiss MrKevinWeiss added this to the Release 2024.04 milestone Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: examples Area: Example Applications Area: Rust Area: Rust wrapper Area: sys Area: System Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: run tests If set, CI server will run tests on hardware 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