Skip to content

patches: link the rules_rust MSVC stdlib patch to upstream (closes #5169) - #6629

Merged
TrapsterDK merged 1 commit into
mainfrom
dx/issue-5169-rules-rust-upstream-link
Nov 4, 2026
Merged

TrapsterDK merged 1 commit into
mainfrom
dx/issue-5169-rules-rust-upstream-link

Conversation

@TrapsterDK

@TrapsterDK TrapsterDK commented Nov 4, 2026 •

Copy link
Copy Markdown
Collaborator

Closes #5169

The rules_rust MSVC stdlib link-flag patch is applied in-repo, and the upstream fix is open at bazelbuild/rules_rust#4376. This change names that PR in the rules_rust row of patches/README.md, the same way the windows_support row links hermeticbuild/windows_support#19, so whoever bumps rules_rust can find the patch and see which upstream release retires it.

Change: one cell in patches/README.md. No code or lockfile change.

Follow-up (not in this PR): once a BCR release of rules_rust carries #4376, delete patches/rules_rust/msvc_stdlib_linkflags.patch and its references (MODULE.bazel, patches/rules_rust/BUILD.bazel, cli/adopt, cli/cli, and the README row).

Checks: docs-only change. tools/dx check --apply --affected ran but stopped at bazel build //:dx with a pre-existing dx_crates lockfile digest mismatch on origin/main (Cargo.toml changed in 7a8789c without a repin). That is unrelated to this change; the diff touches only patches/README.md. Not run: tools/dx test (no test covers a README row), and no nested-Bazel or *_consumer_test targets.

🤖 Generated with Claude Code

Assisted by Claude.

)

Name bazelbuild/rules_rust#4376 in the rules_rust row so the patch is
tied to the upstream PR. Whoever bumps rules_rust drops the patch once
a BCR release carries that PR.

Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011u9TXHDZrfmzFQ2WT8QAKK
@TrapsterDK
TrapsterDK merged commit e1ee57f into main Nov 4, 2026
@TrapsterDK
TrapsterDK deleted the dx/issue-5169-rules-rust-upstream-link branch November 4, 2026 21:39
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.

upstream: rules_rust PR for MSVC stdlib link flags (kernel32.lib, ntdll.lib)

1 participant