Skip to content

fix(build): detect Windows shells in Makefile outputs#2051

Open
Alix-007 wants to merge 2 commits intosipeed:mainfrom
Alix-007:fix/issue-2050-mingw-build
Open

fix(build): detect Windows shells in Makefile outputs#2051
Alix-007 wants to merge 2 commits intosipeed:mainfrom
Alix-007:fix/issue-2050-mingw-build

Conversation

@Alix-007
Copy link
Contributor

@Alix-007 Alix-007 commented Mar 26, 2026

Summary

  • detect MINGW/MSYS/CYGWIN uname -s values as Windows in the top-level Makefile
  • emit .exe output names and Windows-friendly symlink aliases for both build and build-launcher targets
  • keep Linux/macOS build naming behavior unchanged

Validation

  • make -n build UNAME_S=MINGW64_NT-10.0-19045 UNAME_M=x86_64
  • make -n build-launcher UNAME_S=MINGW64_NT-10.0-19045 UNAME_M=x86_64
  • make -n build UNAME_S=Linux UNAME_M=x86_64

Closes #2050

@sipeed-bot sipeed-bot bot added type: bug Something isn't working domain: build labels Mar 26, 2026
@Alix-007
Copy link
Contributor Author

CI is now green on head . The patch is limited to the Windows shell detection fix the issue describes; a maintainer review whenever convenient would be appreciated so we can move this into passive watch.

@Alix-007
Copy link
Contributor Author

CI is now green on head c5fba80cd29349364b23921fe6a5a6d66d7a5800. The patch is limited to the Windows shell detection fix the issue describes; a maintainer review whenever convenient would be appreciated so we can move this into passive watch.

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

Labels

domain: build type: bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Makefile does not support Windows (Git Bash / MSYS2) build

1 participant