Skip to content

Upgrade dependencies and build config#2205

Open
rejuvenile wants to merge 1 commit intoTraceMachina:mainfrom
rejuvenile:pr/01-upgrade-deps
Open

Upgrade dependencies and build config#2205
rejuvenile wants to merge 1 commit intoTraceMachina:mainfrom
rejuvenile:pr/01-upgrade-deps

Conversation

@rejuvenile
Copy link

@rejuvenile rejuvenile commented Mar 12, 2026

Summary

  • Update crate dependencies to latest compatible versions
  • Adjust build configuration for improved compatibility

Test plan

  • cargo check passes
  • CI checks pass

🤖 Generated with Claude Code


This change is Reviewable

Update tonic, prost, and other dependencies to latest versions.
Regenerate protobuf bindings. Add aws-lc-rs and rayon support
to digest hasher. Update BUILD.bazel files for LRE toolchain.

Co-Authored-By: Claude <noreply@anthropic.com>
@CLAassistant
Copy link

CLAassistant commented Mar 12, 2026

CLA assistant check
All committers have signed the CLA.

@palfrey
Copy link
Member

palfrey commented Mar 12, 2026

NativeLink Contributor seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

The commit appears to have been done with the email contributor@nativelink.dev. @rejuvenile, these should be under your own email and you'll need to sign the CLA once that's fixed. Or you may well have already under your email, but we'll find that out once you've done so.

rustls-pki-types = { version = "1.13.1", features = [
"std",
], default-features = false }
socket2 = { version = "0.5.10", default-features = false }
Copy link
Member

Choose a reason for hiding this comment

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

Why was this added?

Copy link
Member

Choose a reason for hiding this comment

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

The items in here don't really have anything to do with the upgrades and should be fished out as a separate commit/PR

Copy link
Member

Choose a reason for hiding this comment

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

Were these changes made by just editing, or by regenerating with bazel run nativelink-proto:update_protos? We'll find out either way TBH once the tests are running

lock.publish(kind, field_metadata)
}
}

Copy link
Member

Choose a reason for hiding this comment

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

What needs this adding?

Copy link
Member

Choose a reason for hiding this comment

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

These should all be reset, as these changes shouldn't be there

[profile.release]
lto = true
lto = "thin"
codegen-units = 16
Copy link
Member

Choose a reason for hiding this comment

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

This isn't a upgrade, it's a change. Has this been tested to see if it improves things?

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.

3 participants