Skip to content

Windows portability: verify the full suite on CI (#11)#15

Merged
freshtonic merged 3 commits into
mainfrom
windows-portability
Jul 25, 2026
Merged

Windows portability: verify the full suite on CI (#11)#15
freshtonic merged 3 commits into
mainfrom
windows-portability

Conversation

@freshtonic

Copy link
Copy Markdown
Owner

Verifying git-queue on real Windows before shipping a Windows binary (#11).

  • Route editor command paths through shell_path() (forward slashes) so git's bundled sh handles them.
  • Windows CI now runs the full test suite (integration + view), not just --lib.

Draft until Windows CI is green; then I'll add the Windows target + PowerShell installer to dist.

… shell

git runs GIT_SEQUENCE_EDITOR/GIT_EDITOR through a POSIX shell (its bundled sh on
Windows), which accepts forward slashes but mishandles backslash separators.
Route every editor command path (cp-based todo/message editors and the
current_exe reorder/stamp editors) through a shell_path() helper that normalises
separators. No-op on Unix. Also flip the Windows CI job from --lib-only to the
full test suite so the git-driven rebase machinery is actually exercised.
The one Windows failure was a test artifact, not a product bug: git-for-windows
defaults core.autocrlf=true, so a checked-out `v2\n` file reads back as
`v2\r\n` and a working-tree byte assertion failed. Pin autocrlf/eol in the
hermetic repo setup so disk-content assertions hold on every platform.
Windows portability is now verified end-to-end (full test suite green on
windows-latest / git 2.55, 0 skips), so add x86_64-pc-windows-msvc to dist
targets and the powershell installer. Release now produces
git-queue-x86_64-pc-windows-msvc.zip and git-queue-installer.ps1.

Docs: website Install section shows the PowerShell installer for Windows and
includes Windows in the prebuilt-binary card; drops the 'source-only, #11'
note. README documents the PowerShell installer. Closes #11.
@freshtonic
freshtonic marked this pull request as ready for review July 25, 2026 06:46
@freshtonic
freshtonic merged commit 7e855c3 into main Jul 25, 2026
8 checks passed
@freshtonic
freshtonic deleted the windows-portability branch July 25, 2026 06:46
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