Skip to content

fix: test Windows LNK1181 LIBPATH quoting fix - #412

Closed
avrabe wants to merge 2 commits into
mainfrom
fix/test-windows-libpath-quoting
Closed

avrabe wants to merge 2 commits into
mainfrom
fix/test-windows-libpath-quoting

Conversation

@avrabe

@avrabe avrabe commented Mar 21, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Combines two fixes for the persistent Windows LNK1181 error:

  1. LIBPATH quoting (MODULE.bazel): Point at avrabe/rules_rust fix/windows-libpath-quoting branch which quotes paths with spaces in bootstrap_process_wrapper.bat
  2. Short output_base (ci.yml): Use --output_base=D:\_b to avoid Windows MAX_PATH (260 char) limit — the rlib paths are ~250 chars with the default output_base

Test plan

  • Builds pass locally (macOS)
  • Key test: Test on windows-latest passes with both fixes combined
  • All other CI checks pass

🤖 Generated with Claude Code

Point git_override at avrabe/rules_rust fix/windows-libpath-quoting branch
(commit 732695b6) which quotes LIBPATH paths containing spaces in the
bootstrap_process_wrapper.bat script. This should fix the persistent
Windows CI failure: LNK1181 cannot open input file librustc_std_workspace_alloc.rlib

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@avrabe

avrabe commented Mar 21, 2026

Copy link
Copy Markdown
Contributor Author

Windows LIBPATH quoting fix doesn't fully resolve LNK1181. Two separate issues found:

  1. BCR test: Files\Microsoft.obj — space-in-path splitting in MSVC lib paths. Closer to what the fix targets but bat script quoting isn't sufficient.
  2. Full test: librustc_std_workspace_alloc.rlib — sysroot rlib not found. May be a symlink/sandbox issue, not path quoting.

Closing to iterate on the fork. Will reopen when the fix covers both code paths.

@avrabe avrabe closed this Mar 21, 2026
Combine both fixes for Windows LNK1181:
1. rules_rust fork with LIBPATH quoting (MODULE.bazel)
2. Short output_base D:\_b to avoid MAX_PATH limit (ci.yml)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@avrabe avrabe reopened this Mar 21, 2026
@avrabe

avrabe commented Mar 21, 2026

Copy link
Copy Markdown
Contributor Author

Not needed — the LNK1181 was purely a MAX_PATH issue, fixed by short output_base in #413. The LIBPATH quoting fix isn't required.

@avrabe avrabe closed this Mar 21, 2026
@avrabe
avrabe deleted the fix/test-windows-libpath-quoting branch May 20, 2026 04:25
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