ci: add native Windows build workflow to verify documented prerequisites#19074
ci: add native Windows build workflow to verify documented prerequisites#19074
Conversation
Adds a workflow that restricts PATH to only tools from the documented prerequisites (Go, Git for Windows, chocolatey cmake/make/mingw) to verify that the docs are sufficient for building on a fresh Windows install. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The chocolatey mingw package installs gcc to C:\ProgramData\mingw64\ rather than the expected choco lib path. Use `which` to discover actual binary locations before restricting PATH. Also fix Git for Windows paths: use /usr/bin and /mingw64/bin directly instead of computing from `which git` (which returns an MSYS2-internal path). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Rewrite the PATH restriction in the native Windows build workflow to use cygpath -w for Windows-native paths (semicolon-separated, C:\... style). MSYS2 bash auto-translates these, so existing bash steps are unaffected. Add a PowerShell build step (shell: pwsh) to verify that make erigon works from the default Windows shell experience.
Install cmake and mingw alongside make, add mingw to PATH, verify tool availability, and build erigon individually before make all. This brings the main CI Windows job in line with the dedicated native build workflow.
|
I can't actually tell if this is identifying the issue that @bloxster reported to me. I'll leave it in draft while I am tracking down the issues. |
|
Git-Bash_full_log.txt No warning at all if compiling wmake.ps1 in file explorer or Powershell (old method). |
See also #17214. |
|
@copilot rebase to main |
|
@anacrolix I've opened a new pull request, #19169, to work on those changes. Once the pull request is ready, I'll request review from you. |
Fixes warning in log at #19074 (comment) Fixes #17214 (GCC most likely). I'm expecting the strlen warning to be fixed, and if the top-level warning thing for cc1 still exists, I'll add a fix for that in this PR. Need to read the build log for a Windows workflow to confirm.
|
I don't like the direction this workflow is going, I think it's too complicated. Also the source of the warning is probably fixed, so I made #19201. |
Summary
Test plan
Native Windows build (docs verification)workflow passes🤖 Generated with Claude Code