Repository navigation
Conversation
The Windows ignore_dirs list documented what Win32 GNU coreutils 8.32 ships, not what this repository cannot build. Three tools that were listed — ls, sync and timeout — compile and pass their tests on Windows, so the Windows CI job never built them at all, which is how the timeout bug that only broke Linux and macOS survived until someone diagnosed vlang#237. The list is now what this repository genuinely cannot build on each platform, so those three are removed. The df, dir, vdir, install, nice and chmod ports also build on Windows and are removed too, but they only exist on open PRs, so they will not be built until those merge; that is a sequencing consequence rather than a mistake, and it is stated here so nobody reads this commit as claiming six more tools are covered on Windows today.
2342e1a to
04db685
Compare
|
Dropped the Both this branch and #250 added that line to the per-platform exclusion map in Vinix is not this PR's subject — the change here is about the Windows exclusion list, where tools this repo already builds and tests for Windows were being skipped. Adding a platform entry is #250's job and it carries the reasoning for why only So this branch's diff is now only the exclusion-list change it was always about, If #250 lands first this branch rebases to a no-op on that file. If this lands first, #250's commit still supplies the |
The Windows
ignore_dirslist documented what Win32 GNU coreutils 8.32 ships, not what this repository cannot build. Three tools that were listed —ls,syncandtimeout— compile and pass their tests on Windows, so the Windows CI job never built them at all, which is how thetimeoutbug that only broke Linux and macOS survived until someone diagnosed #237.The list is now what this repository genuinely cannot build on each platform, so those three are removed.
The
df,dir,vdir,install,niceandchmodports also build on Windows and are removed too, but they only exist on open PRs, so they will not be built until those merge. That is a sequencing consequence rather than a mistake, stated here so nobody reads this commit as claiming six more tools are covered on Windows today.Verified: the collective build compiles 76 tools, and
v test src/ls src/sync src/timeoutpasses 4/4 test files.