已合并
fix(package): 排除7个aicpu公共头在pkg_inc/aicpu_common下的重复安装 #857
fix(package): 排除7个aicpu公共头在pkg_inc/aicpu_common下的重复安装 #857
已合并
liu-wei创建于 9月15日
liu-wei
liu-wei成员
9月15日

描述

cmake/package.cmake中通过PATTERN EXCLUDE排除7个aicpu公共头文件不再安装到pkg_inc/aicpu_common/下(已由aicpu_headers_src扁平安装到include/aicpu/),同步从opbase.xml的file_info块中移除这7个文件条目,保持清单与安装规则一致。

修改的7个文件

  • context/common: cpu_context.h, cpu_kernel.h, cpu_kernel_register.h, cpu_types.h
  • context/cpu_proto: cpu_attr_value.h, cpu_tensor.h, cpu_tensor_shape.h

关联的Issue

#419

测试

  • 编译打包PASS
  • 安装PASS(无ERROR: copy file source doesn't exist)
  • 卸载PASS
  • pre-commit Passed

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 liu-wei 的贡献)
liu-weiliu-wei成员
9月15日 创建了 pull request,commit 8125f63e
atomgit-bot
atomgit-bot
9月15日 评论:

变更摘要

该 PR 修复打包过程中 7 个 aicpu 公共头文件在 pkg_inc/aicpu_common/ 下重复安装的问题:这 7 个头已由 aicpu_headers_src 规则扁平安装到 include/aicpu/,因此本 PR 在 cmake/package.cmake 中通过 PATTERN EXCLUDE 排除它们,并同步从 scripts/package/opbase/opbase.xml 的 file_info 清单中移除对应条目,使安装规则与清单保持一致(前序 PR #829 被 revert 后重新提交,编译打包、安装、卸载均已验证通过)。

主要改动

  • cmake/package.cmake 增加排除规则: 在安装 aicpu_common_public 目录的 install(DIRECTORY ...) 中新增 7 个 PATTERN ... EXCLUDE,排除 cpu_attr_value.h、cpu_tensor.h、cpu_tensor_shape.h、cpu_context.h、cpu_types.h、cpu_kernel.h、cpu_kernel_register.h,避免其再次安装到 pkg_inc/aicpu_common/,并添加注释说明这些头已由 aicpu_headers_src 安装到 include/aicpu/。
  • opbase.xml 的 context/common 块移除条目: 从 file_info 中删除 cpu_context.h、cpu_kernel.h、cpu_kernel_register.h、cpu_types.h 4 个文件条目。
  • opbase.xml 的 context/cpu_proto 块移除条目: 从 file_info 中删除 cpu_attr_value.h、cpu_tensor.h、cpu_tensor_shape.h 3 个文件条目,使打包清单与 cmake/package.cmake 的排除规则保持一致。
likedislike
不准确?
atomgit-bot
atomgit-bot
9月15日 评论:

代码审查

✅ 未发现问题

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

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

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

Module Approval Details

module lgtm status approve status
repo-cann/opbase ✅ 宋恺, 王瑞 (2/2) ✅ 宋恺, 王瑞 (2/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
9月15日 将pengjiahao96,zhou-qilong,wangrui_,rxtfeng,loov1,jia0liang,zl_hw,songkai111,pingchuantang,llimwang,gubaocheng设为评审人
CANN-robotCANN-robot成员
9月15日 将pengjiahao96,zhou-qilong,wangrui_,rxtfeng,loov1,jia0liang,zl_hw,songkai111,pingchuantang,llimwang,gubaocheng设为审查人
liu-wei
liu-wei成员
9月15日 评论:

compile

likedislike
liu-weiliu-wei成员
9月15日 预合并成功(commit_id: 91b616c3eecac0601e3971c0930ee34e5e927c87)
CANN-robot
CANN-robot成员
9月15日 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
9月15日 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
9月15日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
9月15日 添加了label:ci-pipeline-passed
liu-weiliu-wei成员
9月15日 修改了pull request 的描述
songkai111成员
9月15日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
9月15日 添加了label:approved
RuiWang_成员
9月15日 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
9月15日 添加了label:lgtm
CANN-robotCANN-robot成员
9月15日 合入了pull request