Skip to content

Conversation

chrysn
Copy link
Member

@chrysn chrysn commented Jan 24, 2023

Contribution description

As riot-wrappers has advanced a bit since it was last released, Rust modules are switched to using it from git again. (This is a regular ping-pong between testing the latest release in RIOT master, and using released support crates when they're current).

This primarily updates riot-wrappers, which has accumulated several compatible changes. Several other crates receive updates as well.

Testing procedure

  • Green CI

Issues/PRs references

Changes on the riot-wrappers side:

[edit: added:]

This also serves to help preparing a 0.8.1 release of riot-wrappers, which performs some deprecations so that a breaking 0.9 change can be done more effortlessly later on.

@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 Area: Rust Area: Rust wrapper labels Jan 24, 2023
@github-actions github-actions bot added Area: examples Area: Example Applications Area: sys Area: System Area: tests Area: tests and testing framework labels Jan 24, 2023
@riot-ci
Copy link

riot-ci commented Jan 24, 2023

Murdock results

✔️ PASSED

3779abc rust: Update riot-wrappers

Success Failures Total Runtime
6796 0 6796 07m:23s

Artifacts

@chrysn chrysn force-pushed the rust-update-20230124 branch from 54fdba0 to 402fd69 Compare January 24, 2023 18:56
@chrysn
Copy link
Member Author

chrysn commented Jan 24, 2023

An instasquashed commit fixes that, once again, the coap-handler-implementations' fuzziness when it comes to supporting minicbor. (It supports "any" minicbor version, but unless the user picks the same one consistently in all transitive dependencies, things do fall apart).

After the round of updates, that second change was created using cargo update --package='minicbor@0.15.0' --precise '0.13.0' in the rust-gcoap example.

This primarily updates riot-wrappers, which has accumulated several
compatible changes.
@chrysn chrysn force-pushed the rust-update-20230124 branch from 402fd69 to 5981079 Compare January 24, 2023 19:13
@chrysn chrysn requested a review from kaspar030 January 24, 2023 19:37
@chrysn
Copy link
Member Author

chrysn commented Jan 25, 2023

Given I've merged a few more PRs on riot-wrappers today, this was also updated to

@benpicco
Copy link
Contributor

bors merge

1 similar comment
@benpicco
Copy link
Contributor

benpicco commented Feb 1, 2023

bors merge

@bors
Copy link
Contributor

bors bot commented Feb 1, 2023

Build succeeded:

@bors bors bot merged commit cf2d66e into RIOT-OS:master Feb 1, 2023
@chrysn chrysn deleted the rust-update-20230124 branch February 1, 2023 13:40
@MrKevinWeiss MrKevinWeiss added this to the Release 2023.04 milestone Apr 25, 2023
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 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