-
Notifications
You must be signed in to change notification settings - Fork 327
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
chore: bump rust to 1.84 #3469
Conversation
…be27b3fbbc4d7bed681ec8486f8b1d Image tag: b3869ba20c4e9ba2915be302f5b5a4b23da0545af2f1cb896441055372950389
Run URL: https://github.com/dfinity/ic/actions/runs/12806865549 New container image: |
There was a problem hiding this 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!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @DSharifi!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MR LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great ! Thank you!
No canister behavior changes.
There was a problem hiding this 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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, thanks!
This PR upgrades the rust version of the repo to 1.84.
rules_rust
has been upgraded from 0.54.1 to 0.56.0. This is necessary as rust 1.84 removed the targetwasm32-wasi
and renamed it towasm32-wasip1
.rules_ruest
versions pre0.55.0
referencewasm32-wasi
which as a consequence leads to build error. https://github.com/bazelbuild/rules_rust/releases/tag/0.55.0compile_data
flag for data that is only needed at compile time. This is a change that was added in rules_rust0.55.0
. See makedata
runtime-only to align with ecosystem norms bazelbuild/rules_rust#2977.//rs/nns/governance:governance-canbench_test
has also been updated as it failed on CI.ic-ledger-suite-orchestrator-canister.wasm.gz
has been increased from17
mb to18
mb.