已合并
【feat】: 升级cann-cmake至master-044,切换add_cann_sign_file/add_cann_device_project公共函数 #4161
【feat】: 升级cann-cmake至master-044,切换add_cann_sign_file/add_cann_device_project公共函数 #4161
已合并
yelongjian创建于 23 天前
yelongjian成员
23 天前

Pull Request

描述

【PR】: 升级cann-cmake至master-044,切换add_cann_sign_file/add_cann_device_project公共函数

变更类型

请选择本次引入的变更类型:

关联的Issue

如何测试

描述测试此变更的步骤和前提条件:
1.
2.

核对清单

  • [x]我在标题中使用了合适的类型标签(如:feat:, fix:

其他信息

在此添加任何其他关于本次 PR 的说明。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 yelongjian 的贡献)
Yyelongjian成员
23 天前 添加了label:enhancement
Yyelongjian成员
23 天前 创建了 pull request,commit dd7b3a0e
atomgit-bot
atomgit-bot
23 天前 评论:

变更摘要

本次 PR 主要将 cann-cmake 依赖从 master-042 升级至 master-044 版本,并在项目中切换使用新版提供的 add_cann_sign_file / add_cann_device_project 公共函数,同时清理了构建脚本中已不再需要的 VERSION_INFO 参数传递。

主要改动

  • 升级 cann-cmake 版本: cmake/fetch_cann_cmake.cmakeCANN_CMAKE_TAGmaster-042 升至 master-044CANN_CMAKE_SHA256 校验值同步更新,确保下载的依赖包完整性
  • 移除 VERSION_INFO 构建参数: build.sh 中不再向 CMake 传递 -D VERSION_INFO=${VERSION_INFO},对应新版 cann-cmake 中该参数已被废弃或内置处理
likedislike
atomgit-bot
atomgit-bot
23 天前 评论:

代码审查

经过对全部 5 个变更文件的逐项审查,包括对上下文、调用关系的深入排查:

  • CMakeLists.txt:将手动的 ExternalProject_Add(udf_device ...) 替换为 add_cann_device_project(dflow-executor),依赖目标从 udf_device 改为 cann_device,移除了显式的 install(FILES ...)(由新函数内部处理)。对应 version.cmake 的引入使得不再需要手动传递 -DVERSION_INFO
  • build.sh:移除了不再被 CMake 顶层使用的 -D VERSION_INFO=${VERSION_INFO}VERSION_INFO 仍保留在 shell 层用于输出包重命名,一致且无副作用。
  • cmake/device/CMakeLists.txt:新增 include(${GE_DIR}/version.cmake),在 init_cann_project() 之后、add_subdirectory 之前引入,确保 CANN_VERSION_dflow-executor_VERSION 等变量在设备子构建中可用。路径计算正确。
  • cmake/fetch_cann_cmake.cmake:Tag 从 master-042 升级至 master-044,SHA256 同步更新。这是升级依赖的常规操作。
  • dflow/udf/cmake/device/CMakeLists.txt:两处 add_cann_sign_file 调用均增加了 VERSION "${CANN_VERSION_dflow-executor_VERSION}",变量来源为 version.cmakeset_cann_package(dflow-executor VERSION "9.1.0") 所设定,调用链完整。

所有变更逻辑一致,未发现正确的、安全的或可靠性方面的回归问题。


审查总结

优先级 数量
P0 0
P1 0
P2 0
P3 0

各文件审查结论

  • CMakeLists.txt:无问题
  • build.sh:无问题
  • cmake/device/CMakeLists.txt:无问题
  • cmake/fetch_cann_cmake.cmake:无问题
  • dflow/udf/cmake/device/CMakeLists.txt:无问题

整体风险评估:本次变更为常规重构——升级 cann-cmake 依赖版本并切换至公共函数 add_cann_device_project / add_cann_sign_file(增加 VERSION 参数),变更范围清晰、链路上下游一致,风险较低。

⚠️ 已识别出整体风险,但无法提取行内评论,请参考整体评估。

likedislike
CANN-robotCANN-robot成员
23 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
23 天前 评论:

CLA Signature Pass

yelongjian, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
CANN-robot
CANN-robot成员
23 天前 评论:

Thanks for your pull-request.
The full list of commands accepted by me can be found at here
You can get sig-info at here


PR Approval Progress

Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
repo-cann/ge kobemini, zhanj, lining23666 (3/2) zhanj (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
CANN-robotCANN-robot成员
23 天前 将yangyongqiang0606,kobemini,tangqunzhang,xchu42,hugo111,lining23666,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为评审人
CANN-robotCANN-robot成员
23 天前 将yangyongqiang0606,kobemini,tangqunzhang,xchu42,hugo111,lining23666,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为审查人
yelongjian成员
23 天前 评论:

compile

likedislike
Yyelongjian成员
23 天前 update merge request[project id: 8824148, iid: 4161, commit_id: cc51f87b773f10eb210353198a183eb22e6979c6] virtual merging success
CANN-robotCANN-robot成员
23 天前 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
23 天前 评论:

流水线任务触发成功
任务链接 [baf79ca7a1f44788851407f8a80e0db1][流水线指导]

任务名称状态日志下载链接
pre_comment ✅ SUCCESS >>>>>
Compile_X86_compiler ✅ SUCCESS >>>>> >>>>>
Compile_X86_executor ✅ SUCCESS >>>>> >>>>>
Compile_X86_dflow ✅ SUCCESS >>>>> >>>>>
UT_Test_dflow ✅ SUCCESS
UT_Test_ge_common ✅ SUCCESS
UT_Test_parser ✅ SUCCESS
UT_Test_python ✅ SUCCESS >>>>>
UT_Test_rt ✅ SUCCESS
UT_Test_acl ✅ SUCCESS
UT_Test_autofuse ✅ SUCCESS
UT_Test_executor ✅ SUCCESS
UT_Test_autofuse_ascendc_api ✅ SUCCESS
ST_Test_dflow ✅ SUCCESS
ST_Test_ge_common ✅ SUCCESS
ST_Test_parser ✅ SUCCESS
ST_Test_python ✅ SUCCESS >>>>>
ST_Test_rt ✅ SUCCESS
ST_Test_autofuse ✅ SUCCESS
ST_Test_executor ✅ SUCCESS
ST_Test_hetero ✅ SUCCESS
ST_Test_autofuse_ascendc_api ✅ SUCCESS
ST_Test_autofuse_e2e ✅ SUCCESS
UT_Test_fe ✅ SUCCESS
UT_Test_tefusion ✅ SUCCESS
ST_Test_fe ✅ SUCCESS
ST_Test_tefusion ✅ SUCCESS
ST_Test_ge_common_atc ✅ SUCCESS
UT_Test_Report_graphengine ✅ SUCCESS >>>>>
UT_Test_Report_ge_executor_c ✅ SUCCESS >>>>>
UT_Test_Report_ge_autofuse ✅ SUCCESS >>>>>
ST_Test_Report_graphengine ✅ SUCCESS >>>>>
ST_Test_Report_ge_executor_c ✅ SUCCESS >>>>>
ST_Test_Report_ge_autofuse ✅ SUCCESS >>>>>
UT_Test_Report_fe ✅ SUCCESS >>>>>
ST_Test_Report_fe ✅ SUCCESS >>>>>

[2026-07-28 09:50:24]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
23 天前 评论:

流水线任务触发成功
任务链接 [125483b888154c6ea293e209290645ee][流水线指导]

任务名称状态日志下载链接
pre_comment ✅ SUCCESS >>>>>
Compile_ARM_compiler ✅ SUCCESS >>>>> >>>>>
Compile_ARM_executor ✅ SUCCESS >>>>> >>>>>
Compile_ARM_dflow ✅ SUCCESS >>>>> >>>>>
API_Check ✅ SUCCESS >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>

[2026-07-28 09:43:04]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
23 天前 评论:

流水线任务触发成功
任务链接 [5fbb2ae9261a4bc990d27eebb75a6404][流水线指导]

任务名称状态日志下载链接
codecheck_Pr ✅ SUCCESS
antipoison ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
codecheck_dt ✅ SUCCESS >>>>>
codecheck_precommit ✅ SUCCESS >>>>>
StaticCheck_codespell ✅ SUCCESS
StaticCheck_link_validity ✅ SUCCESS
StaticCheck_resource_existence ✅ SUCCESS
StaticCheck_tag_closed ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS

[2026-07-28 09:40:51]    CI执行结束

likedislike
CANN-robotCANN-robot成员
23 天前 添加了label:api-check-pass
CANN-robotCANN-robot成员
23 天前 删除了label:api-check-pass
CANN-robotCANN-robot成员
23 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
23 天前 添加了label:ci-pipeline-passed
lining23666成员
23 天前 评论:

/lgtm

likedislike
GengChao
GengChao成员
23 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
23 天前 添加了label:lgtm
zhanj
zhanj成员
23 天前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
23 天前 添加了label:approved
CANN-robotCANN-robot成员
23 天前 合入了pull request