Skip to content

fix(release): use correct micro.sfx filename on Windows#172

Merged
jasdeepkhalsa merged 1 commit intomasterfrom
fix/releases-for-windows
Apr 3, 2026
Merged

fix(release): use correct micro.sfx filename on Windows#172
jasdeepkhalsa merged 1 commit intomasterfrom
fix/releases-for-windows

Conversation

@jasdeepkhalsa
Copy link
Copy Markdown
Member

fix(release): correct micro.sfx filename on Windows

Problem

The "Combine PHP + PHAR → binary (Windows)" step failed with:

Could not find file 'D:\a\DBDiff\DBDiff\buildroot\bin\micro.sfx.exe'

Root cause

SPC's WindowsBuilder::deploySAPIBinary() copies the built micro SAPI to
buildroot\bin\micro.sfx on all platforms — the filename has no .exe
extension, even on Windows. Both release.yml and release-dry-run.yml
incorrectly referenced micro.sfx.exe.

Confirmed by the SPC docs:

"micro: The build result is buildroot/bin/micro.sfx"

And by WindowsBuilder.phpdeploySAPIBinary() deploys micro.sfx (no .exe) regardless of platform.

Fix

Changed buildroot\bin\micro.sfx.exebuildroot\bin\micro.sfx in the
Windows combine step of both workflows.

Files changed

  • .github/workflows/release.yml
  • .github/workflows/release-dry-run.yml

SPC deploys the micro SAPI to buildroot/bin/micro.sfx on all platforms
including Windows — the filename has no .exe extension. The Windows
combine step was referencing micro.sfx.exe which does not exist.

Fixes: Combine PHP + PHAR → binary (Windows) step failure.
@github-actions github-actions bot added the bug label Apr 3, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 3, 2026

@jasdeepkhalsa jasdeepkhalsa merged commit dd46c7a into master Apr 3, 2026
68 checks passed
@jasdeepkhalsa
Copy link
Copy Markdown
Member Author

Merged quickly due to workflow only changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant