已合并
fix: align run package version with devkit #136
Hyunbin创建于 8月26日
fix: align run package version with devkit #136
已合并
Hyunbin创建于 8月26日
Hyunbin
Hyunbin成员
8月26日

Was This PR authored or co-authored using generative AI tooling?

[x] No
[ ] Yes
__1. AI Agent :
__2. AI Model :
__3. Prompt :

What this PR does / why we need it:

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

Change Type / 改动类型

Testing / 测试信息

Checklist / 检查清单

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 Hyunbin 的贡献)
HyunbinHyunbin成员
8月26日 创建了 pull request,commit 8cb70491
atomgit-bot
atomgit-bot
8月26日 评论:

变更摘要

本 PR 旨在让 run 包(asc-comm 打包产物)的版本与 devkit 对齐:将包版本的来源从默认的 1.0.0 或可配置参数改为统一从新增的根目录 version.cmake 读取(当前 ASCCOMM_VERSION 为 9.2.0),同时把打包脚本参数 --cann-path 统一为 --cann_path,并在安装脚本中新增与 devkit 行为一致的 --install-for-all 选项及权限校验,最后同步更新打包测试。

主要改动

  • 包版本来源统一到 version.cmake: 在 scripts/package/build_package.sh 中,PACKAGE_VERSION 不再默认 1.0.0,新增 read_project_version() 函数从根目录 version.cmake 解析 ASCCOMM_VERSION 作为包版本,移除 --package-version 参数,并将 version.cmake 加入 PACKAGE_INPUT_PATHS 随包发布。
  • 参数命名对齐: build.sh 调用打包脚本时以及 scripts/package/build_package.sh 的参数解析、帮助与错误提示中,--cann-path 统一改为 --cann_path。
  • 新增 --install-for-all 安装选项: scripts/package/install.sh 新增 --install-for-all 参数及 check_install_for_all() 校验函数,要求 CANN 根目录权限(other 位为 5 或 7)支持所有用户读取和遍历,否则安装失败;scripts/package/help.info 同步补充说明并调整描述。
  • 打包测试更新: tests/package/test_run_package.sh 新增断言,校验 version.info 中的 PackageVersion/Version 与 version.cmake 一致,并覆盖 --install-for-all 成功(目录权限变为 555)与受限权限失败场景,同时将 Hcomm 相对 include 检查合并到统一安装布局校验中。
likedislike
不准确?
atomgit-bot
atomgit-bot
8月26日 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
8月26日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
8月26日 评论:

CLA Signature Pass

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

likedislike
此处折叠了57条消息 查看更多
suqwe
suqwe成员
27 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
27 天前 添加了label:lgtm
mlidongfeng
mlidongfeng成员
26 天前 评论:

/approve

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