Skip to content

fix: 修复重构后的 macOS 与 Linux 编译错误 - #89

Merged
shaohaozhi286 merged 3 commits into
masterfrom
codex/refactor-phase1-windows
Sep 29, 2026
Merged

shaohaozhi286 merged 3 commits into
masterfrom
codex/refactor-phase1-windows

Conversation

@shaohaozhi286

@shaohaozhi286 shaohaozhi286 commented Sep 29, 2026 •

Copy link
Copy Markdown
Collaborator

修复一期重构后的 macOS 与 Linux 编译失败,保持现有运行行为:

  • macOS helper 的全局 main 显式调用 acecode::spawn_owned_detached。
  • POSIX 终止信号初始化调用 sigemptyset 时不加 C++ 作用域限定,兼容 macOS SDK 的函数式宏与 Linux 的函数声明。
  • MCP 命令测试在启用、禁用后分别保存策略值快照,再传递局部对象地址,避免对临时对象取地址。

验证:

记录:P4-04。本次收口编译修复;P4-04 的完整矩阵快照和跨平台用例对照仍待验收。人工专项与旧分支迁移继续按既定安排后补。

最终收尾提交仅更新验证记录,产品和测试源码与已通过原生 CI 的 2184247 完全一致。

Qualify the owned worker launcher in the global macOS helper entry point.
Keep separate enabled/disabled policy snapshots in the MCP command test before passing their addresses.

Validation: incremental MSVC Release unit target; 47 BuiltinCommands and invariant tests passed; unchanged 5304-test inventory; strict layers, ownership, documentation paths and OpenSpec; diff check.
Native macOS and Linux CI verification follows this commit.
Avoid a C++ scope qualifier before sigemptyset, which the macOS SDK defines as a function-like macro. Preserve the existing POSIX signal mask behavior.

Validation: incremental Windows unit target; 12 TerminationSignal and invariant tests passed; strict layers and OpenSpec; diff check. Native CI follows.
Linux full CI and all ten package platforms passed on 2184247.
Keep the broader matrix snapshot and inventory acceptance pending.
This commit updates validation records only; product and test sources are unchanged.
@shaohaozhi286
shaohaozhi286 merged commit fbbc699 into master Sep 29, 2026
4 checks passed
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.

2 participants