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

[bazel] update rules_rust to 0.56 #25879

Merged
merged 3 commits into from
Jan 16, 2025
Merged

Conversation

nbdd0121
Copy link
Contributor

This requires some fixes to Rust codebase to keep clippy happy.

@nbdd0121 nbdd0121 requested a review from jwnrt January 15, 2025 13:51
Copy link
Contributor

@jwnrt jwnrt left a comment

Choose a reason for hiding this comment

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

LGTM, thanks. Surprised so much was dropped from the lock file but don't think it's worth working out why

@nbdd0121
Copy link
Contributor Author

rules_rust 0.56 split bindgen/wasm/protobuf into their own extensions and we're only pulling back bindgen.

Fix newly introduced clippy lints in our codebase before updating Rust.

typetag has warnings which is fixed in later versions; this require a
version update to erased-serde.

Signed-off-by: Gary Guo <[email protected]>
This version splits bindgen to a separate extension, so rename
accordingly.

Updating rules_rust_bindgen pulls in a new version of bindgen which
requires a higher Rust version then what we currently use. So update
Rust nightly to the version that latest stable compiler branches from.
Tock needs an older Rust nightly version to compile, so a new repository_set
is used for it instead.

The new version of clippy starts to warn about needless lifetimes in
cases where it's necessary or idiomatic to keep (e.g. trait
implementations), so disable it for now.

Signed-off-by: Gary Guo <[email protected]>
@nbdd0121 nbdd0121 marked this pull request as ready for review January 16, 2025 14:58
@nbdd0121 nbdd0121 requested review from cfrantz and a team as code owners January 16, 2025 14:58
@nbdd0121 nbdd0121 requested a review from pamaury January 16, 2025 15:02
@nbdd0121 nbdd0121 merged commit d515da6 into lowRISC:master Jan 16, 2025
37 of 38 checks passed
@nbdd0121 nbdd0121 deleted the rules_rust branch January 16, 2025 17:35
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