Skip to content

Recognize pinned tools on Windows for 0.29.0-beta.2 - #3

Merged
JeffMusgrave merged 2 commits into
mainfrom
fix/windows-tool-executables
Oct 17, 2026
Merged

JeffMusgrave merged 2 commits into
mainfrom
fix/windows-tool-executables

Conversation

@JeffMusgrave

Copy link
Copy Markdown
Contributor

The v0.29.0-beta.1 release failed on windows-x64: the installed Rust fixture reported the pinned OSV-Scanner as missing. Go reports no execute permission bits for Windows files, so the OSV-Scanner, ty and Python tool checks in policymodule and the shared supply-chain tool check rejected installed tools. Every other host passed and the publish job was skipped, so nothing was released.

  • Accept regular files as executable on Windows in those checks, matching the existing Rust and governed-environment checks.
  • Run the installed native Rust release fixture in the ordinary Windows CI job, so release tags no longer carry its first Windows execution.
  • Prepare 0.29.0-beta.2; the unpublished v0.29.0-beta.1 tag stays immutable.

Verification: policymodule and supplychain unit tests pass on Linux, and both packages vet cleanly for GOOS=windows. The new Windows CI step is the end-to-end check for this fix.

🤖 Generated with Claude Code

JeffMusgrave and others added 2 commits October 16, 2026 23:06
…e in CI

Windows reports no execute permission bits, so the OSV-Scanner, ty and
Python tool checks rejected installed tools and the 0.29.0-beta.1 Windows
release build failed. Run the installed Rust fixture in ordinary Windows CI
so release tags no longer carry its first execution. Prepare 0.29.0-beta.2.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Git for Windows' bin\bash.exe launcher prepends its usr\bin directory, so the
installed Rust fixture linked with coreutils link.exe instead of the linker the
developer shell selected. Launch the fixture with usr\bin\bash.exe, which
inherits PATH unchanged, and verify only the first link.exe on PATH.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@JeffMusgrave
JeffMusgrave merged commit 5bca16e into main Oct 17, 2026
4 checks passed
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