Skip to content

fix(ci): patch rules_rust to fix Windows build PATH overflow#97

Merged
yishuiliunian merged 4 commits intomainfrom
fix/windows-build-path-overflow
Apr 11, 2026
Merged

fix(ci): patch rules_rust to fix Windows build PATH overflow#97
yishuiliunian merged 4 commits intomainfrom
fix/windows-build-path-overflow

Conversation

@yishuiliunian
Copy link
Copy Markdown
Contributor

Summary

Changes

  • patches/BUILD.bazel — new directory, exports patch files
  • patches/rules_rust_windows_consolidate_deps.patch — combined upstream commits eb70659c + 22830c79
  • MODULE.bazelsingle_version_override to apply patch to rules_rust 0.69.0
  • .github/workflows/release.yml — restore Windows (x86_64-pc-windows-msvc) build
  • .github/workflows/ci.yml — add windows-latest to matrix

Test plan

  • Local macOS build passes (bazel build //:loopal)
  • CI passes on all platforms (Linux, macOS, Windows)

Backport upstream commits (eb70659c + 22830c79) as a patch on rules_rust
0.69.0. The process_wrapper now consolidates hundreds of -Ldependency=
paths into a single temp directory via hard links, keeping the PATH env
var under the Win32 32,767-char limit. Re-enable Windows in CI and
Release workflows.
- backend: use platform-specific path for sandbox check test
- runtime: use PathBuf join for platform-agnostic separator in plan test
- system_ipc, meta-hub e2e, agent-hub e2e: skip on Windows (Unix IPC)
@yishuiliunian yishuiliunian merged commit 7a774a1 into main Apr 11, 2026
4 checks passed
@yishuiliunian yishuiliunian deleted the fix/windows-build-path-overflow branch April 11, 2026 11:29
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.

1 participant