已关闭
Add abilityms_src source_set for test targets, drop dead abilityms_static #20363
Add abilityms_src source_set for test targets, drop dead abilityms_static #20363
已关闭
m0_65533854创建于 5 天前关闭于 4 天前
m0_65533854
m0_65533854
5 天前

Close #16134

问题

libabilityms.map 为让测试链接通过,导出了约 100 个类模式;生产消费方(appkit ×2、tools/cc ×1)经审计全部零符号引用(纯 include 路径依赖)。

方案(步骤 1/2,map 收敛随后单独提交)

  1. 僵尸目标 abilityms_static(从未编译,deps 与 so 漂移:缺 modular_object / agent_extension_connection / agent_manager / cli_tool_client / ability_observer_ipc 等)原位替换为 ohos_source_set("abilityms_src")
  2. source_set 与 so 完全对齐:同 sources/configs/include_dirs/deps/external_deps,14 个 feature 条件块逐一比对一致;补 public_configs(cli_tool include)保证测试编译 insight 源码可达;去掉 tdd_config(-flto=full 位码进 host/mac 测试链接有兼容风险)
  3. 750 个测试 BUILD 切换(801 处 dep,test/ + tools/ohos-aa/tests + tools/test/,残留 0)
  4. 生产消费方保持 dep so 不动

影响面

751 文件 +810/-806(绝大部分为单行 dep 切换);so 与 map 本步零改动,生产行为不变。

likedislike
当前Pull Request已关闭, 关闭人@m0_65533854
afwk_helper成员
5 天前 评论:

开始进行AI检视!

AI review has been started, please wait...

likedislike
afwk_helper成员
5 天前 评论:
check type result report
start ai_review pass -
likedislike
m0_65533854m0_65533854
5 天前 修改了pull request 的描述
openharmony_ciopenharmony_ci成员
5 天前 添加了label:waiting_on_author
afwk_helper成员
5 天前 评论:

⚠️ 🤖 AI 代码检视报告 ⚠️

总体评估: NEEDS_ATTENTION

问题统计:

  • 总问题数: 0
  • 严重问题: 0
  • 高危问题: 0

摘要:
PR将僵尸目标abilityms_static替换为abilityms_src source_set并切换750个测试BUILD文件依赖,架构方向合理,但由于未提供实际代码diff和完整文件内容,无法进行逐行深入审查,需补充diff内容后重新评审。

📊 详细报告
查看完整的审查详情,包括具体的问题描述、建议和代码位置:
🔗 查看详细报告


此评论由 OpenHarmony Insight 代码审查系统自动生成

likedislike
此处折叠了138条消息 查看更多
openharmony_ci
openharmony_ci成员
4 天前 评论:

代码门禁通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/6a97f24364650f998bec91c1/runlist

静态检查:

# check type result report
1 codeCheck pass >>>

编译测试:
# Device build result test result package
1 ohos-sdk success NA >>>
2 dayu200 success success >>>
3 dayu200_tdd success NA >>>
4 master_inner_build failed(IGNORE)(联合门禁失败) NA >>>
5 mac-sdk success NA >>>
6 ohos-host_mini_tdd success NA >>>
7 dayu600_7885 success success >>>
8 x86_64_virt success NA >>>

likedislike
openharmony_ciopenharmony_ci成员
4 天前 删除了label:waiting_on_author
openharmony_ciopenharmony_ci成员
4 天前 添加了label:waiting_for_review
openharmony_ci
openharmony_ci成员
4 天前 评论:

您好,Committer @jayleehw @zhangyafei-echo @littlejerry1 @jsjzju @wkljy @yzkp @ccllee1 @yangxuguang-huawei @li-weifeng2024 @wendel ,请分配检视人员检视该PR,可以通过命令"assign [@someone_id]"分配检视人员,也可以直接评论"assign"分配给自己进行检视。


Hello, Committer @jayleehw @zhangyafei-echo @littlejerry1 @jsjzju @wkljy @yzkp @ccllee1 @yangxuguang-huawei @li-weifeng2024 @wendel . Please assign someone to review the PR. You can assign a reviewer by using the command "assign [@someone_id]", or you can comment "assign" to review the PR by yourself.

likedislike
m0_65533854m0_65533854
4 天前 关闭了 pull request