Star725
1026
代码
代码
Issues60
Pull Requests204
Actions
项目讨论
Wiki
分析
Star725
1026
已合并
fix: 修复build.sh参数文档、移除未使用依赖、版本号升级至9.2.0 #4729
DragonBornHD84创建于 7月30日
fix: 修复build.sh参数文档、移除未使用依赖、版本号升级至9.2.0 #4729
已合并
DragonBornHD84创建于 7月30日
从DragonBornHD84/asc-devkitdev合入到cann/asc-devkitmaster
D
DragonBornHD84成员
7月30日

描述

本次改动包含三部分：

  1. 修复 build.sh 参数文档缺失（#1271）：-p/--cann_path 帮助文本仅示例了 root 用户路径，未明示非 root 用户默认为 $HOME/Ascend/cann；-j 未说明并发选择策略。在 --help、--pkg --help、-t --help 三处帮助输出中补充 root/非 root 双默认路径说明，并补充 -j 超过 CPU 核数自动降为核数的策略说明。
  2. 移除 2 项未使用的三方依赖（#1280）：
  • .devcontainer/Dockerfile：移除 gperf，全仓无 libseccomp 依赖，gperf 仅在 libseccomp 源码构建时需要，属无用安装项。
  • CMakeLists.txt / mc2_server_minimal.cmake：移除 rdma-core 三方依赖声明及 rdma_core_headers 相关代码。全仓源码无 infiniband/verbs.h 等 rdma-core 系统头文件引用，源码仅使用项目内部头文件。
  1. 版本号统一升级至 9.2.0：build.sh 的 VERSION_INFO、version.cmake 的 asc-devkit 包版本、hccl/cc/version.cmake 的 hccl 包版本三处统一从 8.5.0/9.1.0/9.0.0 升级为 9.2.0。

关联的Issue

测试

  • bash build.sh --help、bash build.sh --pkg --help、bash build.sh -t --help 三处帮助输出均验证 -p 双路径与 -j 策略展示正确，$HOME 字面量未被展开。
  • bash build.sh -j 16 全量编译验证通过（EXIT_CODE=0），libmc2_server.so 成功生成，构建达 100% 并完成打包，日志无 rdma 相关错误，确认移除 rdma-core 后构建无回归。
  • 版本号改动后重新构建验证通过（EXIT_CODE=0）。

文档更新

  • build.sh：更新 -p/--cann_path、-j 参数帮助说明文本（对应 #1271）。

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
（感谢 DragonBornHD84 的贡献）
DDragonBornHD84成员
7月30日 创建了 pull request，commit 19ed1900
DDragonBornHD84成员
7月30日 关联了issue：[TTFHW] build.sh 中 -p / --cann_path 参数文档未明示默认安装路径（/usr/local/Ascend/cann 仅对 root 用户有效，非 ，[Requirement|需求建议]: [第三方依赖消减] 移除 2 项未使用的三方依赖
atomgit-bot
atomgit-bot
7月30日 评论：

变更摘要

本次 PR 主要完成了三项工作：修复 build.sh 中 -p/-j 参数的帮助文档缺失问题，移除两项未使用的三方依赖（gperf 和 rdma-core），以及将项目版本号统一从旧版本升级至 9.2.0。

主要改动

  • build.sh 帮助文档补充默认路径与并发策略说明：在 --help、--pkg --help、-t --help 三处帮助输出中，为 -p/--cann_path 补充了 root 用户默认 /usr/local/Ascend/cann 与非 root 用户默认 $HOME/Ascend/cann 的双路径说明，并为 -j 补充了超过 CPU 核数时自动降为核数的策略说明。
  • 移除 .devcontainer/Dockerfile 中未使用的 gperf 依赖：从 apt 安装列表中删除 gperf，该项仅在 libseccomp 源码构建时需要，全仓无对应依赖。
  • 移除 mc2_server_minimal.cmake 中未使用的 rdma_core_headers 依赖：在 _add_mc2_server_object 和 configure_mc2_server_minimal_target 两处 target_link_libraries 中删除 rdma_core_headers，全仓源码无 infiniband/verbs.h 等 rdma-core 系统头文件引用。
  • 版本号统一升级至 9.2.0：version.cmake 中 asc-devkit 包版本从 9.1.0 升级为 9.2.0，hccl/cc/version.cmake 中 hccl 包版本从 9.0.0 升级为 9.2.0。
likedislike
atomgit-bot
atomgit-bot
7月30日 评论：

代码审查

审查总结

逐文件审查结果：

文件 结论
.devcontainer/Dockerfile 无问题 — 移除 gperf 干净，全仓无残留引用
build.sh 1 个 P2 问题 — VERSION_INFO 遗漏升级；帮助文本新增正确
impl/adv_api/detail/hccl/cc/src/aicpu_kfc/CMakeLists.txt 无问题 — rdma-core 声明及 rdma_core_headers 库移除完整，无残留引用
impl/adv_api/detail/hccl/cc/src/aicpu_kfc/mc2_server_minimal.cmake 无问题 — 两处 rdma_core_headers 链接引用同步移除
impl/adv_api/detail/hccl/cc/version.cmake 无问题 — hccl 包版本 9.0.0 → 9.2.0
version.cmake 无问题 — asc-devkit 包版本 9.1.0 → 9.2.0

发现问题统计： P0: 0, P1: 0, P2: 1, P3: 0

整体风险评估： 低风险。唯一的 P2 问题是 build.sh 中 VERSION_INFO 遗漏从 8.5.0 升级到 9.2.0，与 PR 描述的版本统一升级意图不一致，会导致 --pkg 打包时产物版本元数据错误。其余 5 个文件的变更均正确、完整。

类型 数量
🔴 阻塞 0
🟡 建议 1

💬 仅评论

likedislike
CANN-robotCANN-robot成员
7月30日 添加了label：cann-cla/yes
CANN-robot
CANN-robot成员
7月30日 评论：

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/asc-devkit ✅ XuebinYang, bluesky901 (2/2) ✅ XuebinYang (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
7月30日 将zengjuan,zhanghao_0689,suqwe,wulinyu,kong0808,dengjing_aoe,YeZZzzz1,bluesky901,yangbinhua,zhudi22,chenyiyuan,xun_zhuge,XuebinYang,sjtulxh,bianque,houyuyang,Lycheeeee,chentianyu19,shi_nanxiang,wuyang_hw,jiangxinyu3,ascendhjh,hujiawen_kaven,wangxiaozhi,louyuxuan,hyolee,yukangchen设为评审人
CANN-robotCANN-robot成员
7月30日 将zengjuan,zhanghao_0689,suqwe,wulinyu,kong0808,dengjing_aoe,YeZZzzz1,bluesky901,yangbinhua,zhudi22,chenyiyuan,xun_zhuge,XuebinYang,sjtulxh,bianque,houyuyang,Lycheeeee,chentianyu19,shi_nanxiang,wuyang_hw,jiangxinyu3,ascendhjh,hujiawen_kaven,wangxiaozhi,louyuxuan,hyolee,yukangchen设为审查人
atomgit-bot
atomgit-bot7月30日进行代码检视1
build.sh
@@ -32,7 +32,7 @@
3232CHANGED_FILES=""
3333CI_MODE=FALSE
3434ENABLE_SIGN="false"
3535VERSION_INFO="8.5.0"
atomgit-bot
atomgit-bot7月30日评论：

🟡 Medium Priority

PR 描述明确声明三处版本号统一升级至 9.2.0："build.sh 的 VERSION_INFO、version.cmake 的 asc-devkit 包版本、hccl/cc/version.cmake 的 hccl 包版本三处统一从 8.5.0/9.1.0/9.0.0 升级为 9.2.0"。但实际 diff 中 build.sh 的第 35 行 VERSION_INFO="8.5.0" 未被修改，仍然是旧值 8.5.0。

该变量在第 815 行通过 -DVERSION_INFO=${VERSION_INFO} 传递给 CMake 打包流程，用于指定生成的包的版本元数据。若保持 8.5.0，打出的包版本号将与另外两处 cmake 声明的 9.2.0 不一致，导致版本元数据错误。

changed line: VERSION_INFO="8.5.0"（第 35 行，未在 diff 中变更）→ 未匹配 PR 声明的升级意图 → 打包时传递错误版本号到 CMake → 产物版本元数据与其他包版本声明不一致。

修复方向：将第 35 行改为 VERSION_INFO="9.2.0"。

建议：将 build.sh 第 35 行的 VERSION_INFO 从 "8.5.0" 改为 "9.2.0"，与 PR 描述中声明的版本统一升级意图保持一致。

改动建议
35
- VERSION_INFO="8.5.0"
35
+ VERSION_INFO="9.2.0"
应用建议
likedislike
D
DragonBornHD84成员
7月30日 评论：

compile

likedislike
DDragonBornHD84成员
7月30日 update merge request[project id: 8547062, iid: 4729, commit_id: d7babe940548994b2e8d9c5676e93f3e1a08c160] virtual merging success
DDragonBornHD84成员
7月30日 解决了最后一个问题
CANN-robotCANN-robot成员
7月30日 添加了label：ci-pipeline-running
CANN-robot
CANN-robot成员
7月30日 评论：

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

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

[2026-07-30 16:52:39]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
7月30日 评论：

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

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_ubuntu24 ✅ SUCCESS >>>>> >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>

[2026-07-30 17:01:29]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
7月30日 评论：

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

任务名称状态日志下载链接
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
UT_Test_adv_test_one ✅ SUCCESS
UT_Test_adv_test_two ✅ SUCCESS
UT_Test_basic_test_one ✅ SUCCESS
UT_Test_basic_test_two ✅ SUCCESS
UT_Test_basic_test_three ✅ SUCCESS
UT_Test_basic_test_four ✅ SUCCESS
UT_Test_basic_test_five ✅ SUCCESS
UT_Test_arm ✅ SUCCESS
API_Check ✅ SUCCESS >>>>>
UT_Test_report ✅ SUCCESS >>>>>

[2026-07-30 17:01:30]    CI执行结束

likedislike
CANN-robotCANN-robot成员
7月30日 添加了label：api-check-pass
CANN-robotCANN-robot成员
7月30日 删除了label：ci-pipeline-running
CANN-robotCANN-robot成员
7月30日 添加了label：ci-pipeline-passed
D
DragonBornHD84成员
25 天前 评论：

compile

likedislike
DDragonBornHD84成员
25 天前 update merge request[project id: 8547062, iid: 4729, commit_id: e720c6fd66fabff0cca94601e366453b0af11a9c] virtual merging success
CANN-robotCANN-robot成员
25 天前 删除了label：ci-pipeline-passedapi-check-pass
CANN-robotCANN-robot成员
25 天前 添加了label：ci-pipeline-running
CANN-robot
CANN-robot成员
25 天前 评论：

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

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

[2026-08-11 15:21:57]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
25 天前 评论：

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

任务名称状态日志下载链接
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
UT_Test_adv_test_one ✅ SUCCESS
UT_Test_adv_test_two ✅ SUCCESS
UT_Test_basic_test_one ✅ SUCCESS
UT_Test_basic_test_two ✅ SUCCESS
UT_Test_basic_test_three ✅ SUCCESS
UT_Test_basic_test_four ✅ SUCCESS
UT_Test_basic_test_five ✅ SUCCESS
UT_Test_arm ✅ SUCCESS
API_Check ✅ SUCCESS >>>>>
UT_Test_report ✅ SUCCESS >>>>>

[2026-08-11 15:31:01]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
25 天前 评论：

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

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_ubuntu24 ✅ SUCCESS >>>>> >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>

[2026-08-11 15:31:57]    CI执行结束

likedislike
CANN-robotCANN-robot成员
25 天前 添加了label：api-check-pass
CANN-robotCANN-robot成员
25 天前 删除了label：api-check-pass
CANN-robotCANN-robot成员
25 天前 删除了label：ci-pipeline-running
CANN-robotCANN-robot成员
25 天前 添加了label：ci-pipeline-passed
B
bluesky901成员
25 天前 评论：

/lgtm

likedislike
X
XuebinYang成员
25 天前 评论：

/approve

likedislike
CANN-robotCANN-robot成员
25 天前 添加了label：lgtmapproved
CANN-robotCANN-robot成员
25 天前 合入了pull request