Skip to content

fix(packaging): refresh root Cargo lockfile - #3267

Merged
johntmyers merged 1 commit into
mainfrom
fix-cargo-lockfile/pm
Sep 11, 2026
Merged

fix(packaging): refresh root Cargo lockfile#3267
johntmyers merged 1 commit into
mainfrom
fix-cargo-lockfile/pm

Conversation

@pimlock

@pimlock pimlock commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

Refresh the root Cargo lockfile so locked dependency resolution and release packaging succeed. The stale lockfile caused cargo vendor --locked to fail in Release Dev.

Cargo restores the ring edge because the Windows prebuilt-z3 feature enables z3-sys's host-side archive downloader. Its dependency path is z3-sys build dependency -> reqwest -> rustls -> ring. OpenShell's target-side TLS configuration still uses AWS-LC, and ring is not linked into the OpenShell binary. The existing exception in deny.toml documents this build-time-only use and restricts it to the rustls and rustls-webpki wrappers.

Related Issue

No issue required: localized lockfile repair for the failed Release Dev run.

Changes

  • Record the missing quinn dependency edge for reqwest
  • Record the missing ring dependency edge for rustls

Testing

  • cargo vendor --locked --quiet succeeds
  • mise run pre-commit passes
  • Unit tests added or updated (not applicable: generated lockfile correction)
  • E2E tests added or updated (not applicable: packaging metadata only)

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)

Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
@johntmyers
johntmyers added this pull request to the merge queue Sep 11, 2026
Merged via the queue into main with commit 35f15d8 Sep 11, 2026
65 checks passed
@johntmyers
johntmyers deleted the fix-cargo-lockfile/pm branch September 11, 2026 01:21
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