Skip to content

emrg: build-release 二十次修复 — smoke Windows cmd 路径 cygpath 转换(rant #13) - #388

Merged
argszero merged 1 commit into
masterfrom
feature/build-release-fixes20
Aug 4, 2026
Merged

emrg: build-release 二十次修复 — smoke Windows cmd 路径 cygpath 转换(rant #13)#388
argszero merged 1 commit into
masterfrom
feature/build-release-fixes20

Conversation

@argszero

Copy link
Copy Markdown
Owner

第 20 次实跑:3/4 平台成功(WINDIR 判断修复生效,POSIX 恢复),Windows Inno 编译通过(artifacts 生成),smoke 第 1 步 ✓,第 2 步 daemon 启动 exit 1(5 秒即挂,短于 15 秒轮询)。

根因cmd //c start 分支的 $HOME 是 Git Bash POSIX 路径(/c/Users/runneradmin/...),cmd 是 Windows 原生程序不认识 POSIX 路径 → start 失败 → daemon 未启动 → exit 1。

修复:cmd 分支用 cygpath -m$HOME/.emrg/install/bin/emrgd.cmd 转 Windows 路径(C:/Users/... 正斜杠形式,cmd 可读)。2 处同步。

验证

  • bash -n OK
  • cygpath -m 转换模拟:/c/Users/runneradmin/... → C:/Users/runneradmin/... ✓
  • POSIX 分支不受影响(无 WINDIR → nohup)✓
  • pytest 464 passed;import checks OK

背景:rant #13 Phase 4 发布期。合并后重打 tag 第 21 次实跑,预期 4/4 全绿 → Release 发布。

第 20 次实跑:3/4 平台成功(WINDIR 判断修复生效,POSIX 恢复),
Windows Inno 编译通过(artifacts 生成),smoke 第 1 步 ✓,第 2 步
daemon 启动 exit 1(5 秒即挂,短于 15 秒轮询)。
根因:cmd //c start 分支的 $HOME 是 Git Bash POSIX 路径
(/c/Users/runneradmin/...),cmd 是 Windows 原生程序不认识
POSIX 路径 → start 失败 → daemon 未启动 → exit 1。
修复:cmd 分支用 cygpath -m 将 $HOME/.emrg/install/bin/emrgd.cmd
转 Windows 路径(C:/Users/... 正斜杠形式,cmd 可读)。2 处同步。
验证:
- bash -n OK
- cygpath -m 转换模拟:/c/Users/runneradmin/... → C:/Users/runneradmin/... ✓
- POSIX 分支不受影响(无 WINDIR → nohup)✓
- pytest 464 passed;import checks OK

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle 20260805-0091

审查内容:smoke-test.sh cmd 分支 cygpath -m 转 Windows 路径($HOME POSIX→C:/...),2 处同步。修第 20 次 Windows daemon 启动 exit 1。

验证:bash -n OK;cygpath 转换模拟正确;POSIX 分支不受影响;pytest 464 passed。根因分析准确(cmd 不认 POSIX 路径)。

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle 20260805-0092

第二次复核:diff 与上轮一致(smoke-test.sh cmd 分支 cygpath -m 转 Windows 路径,2 处)。bash -n OK;转换模拟正确;POSIX 分支不受影响。无新增问题。

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle 20260805-0093

第三次复核:diff 稳定(smoke-test.sh cmd 分支 cygpath -m 转 Windows 路径)。bash -n OK;转换模拟正确;pytest 464 passed。3 个不同 cycle(0091/0092/0093)连续 LGTM,满足合并条件。

@argszero
argszero merged commit f7143e3 into masterAug 4, 2026
1 check passed
@argszero
argszero deleted the feature/build-release-fixes20 branch August 5, 2026 06:18
Sign up for freeto 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

@argszero