已开启
docs: 修正开发验证包头文件范围说明 #228
docs: 修正开发验证包头文件范围说明 #228
已开启
KaranocaVe创建于 10 天前
KaranocaVe
KaranocaVe
10 天前

docs: 修正开发验证包头文件范围说明

README.md 当前将 build.sh --pkg 描述为只包含 AICore 头文件。制包脚本会递归收集 include/ 和 src/ 中的 .h 文件:Hcomm 头文件安装到 adv_api,其余头文件安装到 comm_api。

请更新 README,准确说明该范围,同时保留“不包含 SO”的信息。这是文档修改,不改变包内容或安装行为。

验证:包测试统计四组头文件,并检查 ccu/ccu_host_launch.h 会被打包到 comm_api。

关联 Issue: https://gitcode.com/cann/asc-comm/issues/41

likedislike
合并受阻
KaranocaVeKaranocaVe
10 天前 创建了 pull request,commit ca4871f8
KaranocaVeKaranocaVe
10 天前 关联了issue:[Documentation|文档反馈]: 开发验证包头文件范围说明不准确
atomgit-bot
atomgit-bot
10 天前 评论:

变更摘要

本次变更是一处纯文档修正,目标是让 README.md 中关于 build.sh --pkg 打包范围的描述与实际制包行为一致。原说明将该选项描述为仅包含 AICore 头文件,而制包脚本实际会递归收集 include/ 和 src/ 下的 .h 文件,其中 Hcomm 头文件安装到 adv_api,其余头文件安装到 comm_api。更新后的文档准确反映这一范围,同时继续保留“不包含 SO”的信息;该改动不涉及包内容或安装行为的任何调整。

主要改动

  • 修正 build.sh --pkg 的头文件范围说明: README.md 中不再将该选项描述为仅包含 AICore 头文件,而是说明会递归收集 include/ 与 src/ 中的 .h 文件。
  • 补充头文件安装目录区分: 明确 Hcomm 头文件安装到 adv_api,其余头文件安装到 comm_api。
  • 保留“不包含 SO”的既有说明: 在更新范围描述的同时,继续声明该打包方式不包含 SO。
likedislike
不准确?
atomgit-bot
atomgit-bot
10 天前 评论:

🤖 本次改动均为数据 / 生成 / 二进制文件(如测试用例、锁文件、媒体资源),没有可审查的代码逻辑,已跳过 AI 代码审查。

likedislike
不准确?
CANN-robotCANN-robot成员
10 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
10 天前 评论:

CLA Signature Pass

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

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

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.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.


PR Approval Progress

⚠️ This PR does not yet meet the following requirements:lgtm (requires ≥ 2 person(s) per module)、approve (requires ≥ 1 person(s) per module)

Module Approval Details

module lgtm status approve status
repo-cann/asc-comm ❌ (0/2)(You can also ask: kong0808, 苏建加, 黄金华, 石楠翔, 闫庆尚) ❌ (0/1)(You can also ask: 闫庆尚, zhanghao_0689, dengjing_aoe, 李东锋, jiangxinyu3)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
KaranocaVe
KaranocaVe
10 天前 评论:

/compile

likedislike
KaranocaVeKaranocaVe
10 天前 预合并成功(commit_id: aa5dc07379eab446389f141a0c572fa0ae9358a9)
CANN-robot
CANN-robot成员
10 天前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
10 天前 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
10 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
10 天前 添加了label:ci-pipeline-passed
KaranocaVe
KaranocaVe
7 天前 评论:

/compile

likedislike
KaranocaVeKaranocaVe
7 天前 预合并成功(commit_id: 08c6d2a7b721cf6222d059879d8aa952196762fc)
CANN-robot
CANN-robot成员
7 天前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
7 天前 删除了label:ci-pipeline-passed
CANN-robotCANN-robot成员
7 天前 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
7 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
7 天前 添加了label:ci-pipeline-passed