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

chore: bump rust to 1.84 #3469

Merged
merged 39 commits into from
Jan 20, 2025
Merged

chore: bump rust to 1.84 #3469

merged 39 commits into from
Jan 20, 2025

Conversation

DSharifi
Copy link
Member

@DSharifi DSharifi commented Jan 16, 2025

This PR upgrades the rust version of the repo to 1.84.

  • Code changes to satisfy new clippy or compiler lints.
  • rules_rust has been upgraded from 0.54.1 to 0.56.0. This is necessary as rust 1.84 removed the target wasm32-wasi and renamed it to wasm32-wasip1. rules_ruest versions pre 0.55.0 reference wasm32-wasi which as a consequence leads to build error. https://github.com/bazelbuild/rules_rust/releases/tag/0.55.0
  • Bazel build files have been updated with thecompile_data flag for data that is only needed at compile time. This is a change that was added in rules_rust 0.55.0. See make data runtime-only to align with ecosystem norms bazelbuild/rules_rust#2977.
  • Expecations for //rs/nns/governance:governance-canbench_test has also been updated as it failed on CI.
  • Expected compressed size for ic-ledger-suite-orchestrator-canister.wasm.gz has been increased from 17 mb to 18 mb.

@github-actions github-actions bot added the chore label Jan 16, 2025
…be27b3fbbc4d7bed681ec8486f8b1d

Image tag: b3869ba20c4e9ba2915be302f5b5a4b23da0545af2f1cb896441055372950389
Copy link
Contributor

Run URL: https://github.com/dfinity/ic/actions/runs/12806865549

New container image: sha256:2e8185171700872d48fdfb4b08e175fca5be27b3fbbc4d7bed681ec8486f8b1d
New container tag: b3869ba20c4e9ba2915be302f5b5a4b23da0545af2f1cb896441055372950389

Copy link
Member

@gregorydemay gregorydemay left a comment

Choose a reason for hiding this comment

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

Thanks @DSharifi for this PR!. XC team changes look good!

publish/canisters/BUILD.bazel Show resolved Hide resolved
@gregorydemay gregorydemay requested a review from a team January 20, 2025 12:29
Copy link
Member

@mbjorkqvist mbjorkqvist left a comment

Choose a reason for hiding this comment

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

Thanks @DSharifi!

Copy link
Contributor

@pakhomov-dfinity pakhomov-dfinity left a comment

Choose a reason for hiding this comment

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

MR LGTM

Copy link
Member

@rumenov rumenov left a comment

Choose a reason for hiding this comment

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

Great ! Thank you!

@DSharifi DSharifi dismissed github-actions[bot]’s stale review January 20, 2025 12:54

No canister behavior changes.

@DSharifi DSharifi enabled auto-merge January 20, 2025 13:05
Copy link
Member

@oggy-dfin oggy-dfin left a comment

Choose a reason for hiding this comment

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

tla_instrumentation changes LGTM

Copy link
Contributor

@kpop-dfinity kpop-dfinity left a comment

Choose a reason for hiding this comment

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

Awesome, thanks!

@DSharifi DSharifi added this pull request to the merge queue Jan 20, 2025
Merged via the queue into master with commit fc27870 Jan 20, 2025
30 checks passed
@DSharifi DSharifi deleted the dsharifi/bump-rust-1.84 branch January 20, 2025 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.