已合并
refactor: 切换工程公共打包脚本 #1933
refactor: 切换工程公共打包脚本 #1933
已合并
maohp_hw创建于 4月30日
maohp_hw
maohp_hw成员
4月30日

描述

针对原本使用本地维护的打包脚本(scripts/package/目录下)进行软件包构建。这种设计存在维护成本高、脚本重复、与其他工程不一致的问题。
随着构建系统的演进,现在需要:

  1. 统一工程间的构建打包流程,减少重复维护
  2. 使用工程公共维护的打包脚本,提高构建稳定性
  3. 简化本地构建配置,降低维护成本
    本次重构将本地打包脚本迁移至工程公共维护的CANN公共构建脚本,通过fetch_cann_cmake机制引用,同时保持构建功能不变。
    具体变更:
  4. 删除本地打包脚本:
    • 删除scripts/package/common/目录下所有Python脚本(filelist.py、packer.py、pkg_parser.py、version_info.py等)
    • 删除scripts/package/common/sh/目录下所有Shell脚本(common_func*.inc、install_common_parser.sh等)
    • 删除scripts/package/latest_manager/目录
    • 删除scripts/package/package.py
    • 删除cmake/makeself.cmake
  5. 切换至公共构建脚本:
    • 更新cmake/fetch_cann_cmake.cmake,将CANN_CMAKE_TAG从master-002升级至master-007
    • 添加SHA256校验,确保下载的公共脚本完整性
    • 修改cmake/package.cmake,将脚本引用从本地路径改为${CANN_CMAKE_DIR}公共路径
    • 简化打包配置,移除CPack配置,改用set_cann_cpack_config函数
  6. 调整构建路径:
    • 修改cmake/version_info.cmake,将构建输出路径从${CPACK_CMAKE_BINARY_DIR}改为${CMAKE_BINARY_DIR}

关联的Issue

关联的Issue #605

测试

  • 构建测试正常通过
  • 打包功能验证正常
  • 无功能变更,不影响现有构建流程

文档更新

NA

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 maohp_hw 的贡献)
maohp_hwmaohp_hw成员
4月30日 创建了 pull request,commit 84385e07
CANN-robot
CANN-robot成员
4月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
cmake ✅ yangbinhua, 诸葛洵 (2/2) ✅ 诸葛洵 (1/1)
scripts ✅ 诸葛洵, yangbinhua (2/2) ✅ 诸葛洵 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
4月30日 添加了label:cann-cla/yes
CANN-robotCANN-robot成员
4月30日 将zengjuan,zhanghao_0689,yanqingshang,wulinyu,zhuliangying,kong0808,dengjing_aoe,YeZZzzz1,bluesky901,wqtshg_wt,yangbinhua,gao_dafa,chenyiyuan,xun_zhuge,ookkkkoo1122,bianque,houyuyang,shi_nanxiang,wuyang_hw,ascendhjh,DragonBornHD84,jiangZD设为评审人
CANN-robotCANN-robot成员
4月30日 将gao_dafa,xun_zhuge设为审查人
此处折叠了200条消息 查看更多
yangbinhua
yangbinhua成员
5月9日 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
5月9日 添加了label:lgtm
CANN-robot
CANN-robot成员
5月9日 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: 诸葛洵.
Reviewers who wrote a comment of /lgtm are: 诸葛洵, yangbinhua.

likedislike
CANN-robotCANN-robot成员
5月9日 关闭了关联的issue
CANN-robotCANN-robot成员
5月9日 合入了pull request