已合并
fix: 更新 version.cmake 依赖列表与版本格式 (#142) #381
fix: 更新 version.cmake 依赖列表与版本格式 (#142) #381
已合并
sinat_31531339创建于 6月30日
sinat_31531339
sinat_31531339成员
6月30日

描述

Issue #142 - oam-tools 仓依赖版本号不正确

问题:
version.cmake 依赖列表的版本号未使用语义化版本约束格式。

修复内容:

  1. 版本格式更新

    • "9.0"">=9.0" (语义化版本约束,允许向前兼容)
  2. 包版本号更新

    • set_cann_package(oam-tools VERSION "9.0.0")"9.1.0"
  3. 格式规范

    • 统一使用英文引号(issue 中混用了中文引号,可能导致兼容性问题)

变更类型

请选择本次引入的变更类型(勾选对应项):

关联的Issue

关联 Issue #142

如何测试

  1. CMake 语法检查
  2. pre-commit 检查通过(OAT、codespell)
  3. 云端构建流水线验证依赖解析

核对清单

其他信息

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 sinat_31531339 的贡献)
sinat_31531339sinat_31531339成员
6月30日 关联了issue:[Bug-Report|缺陷反馈]: oam-tools仓依赖版本号不正确
CANN-robotCANN-robot成员
6月30日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
6月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/oam-tools zhuliangying, 金瑛琪 (2/2) zhuliangying (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
atomgit-bot
atomgit-bot
6月30日 评论:

变更摘要

本次 PR 修复了 version.cmake 中依赖项版本号格式不正确以及依赖列表不完整的问题(关联 Issue #142)。主要将固定版本号 "9.0" 统一改为语义化版本约束 ">=9.0",并补全了构建依赖(build dependencies)和运行依赖(run dependencies)中缺失的 ge-executorasc-devkitopbase 以及 ge-compiler 组件。

主要改动

  • 版本约束格式统一:将所有依赖的版本号从固定格式 "9.0" 改为语义化约束 ">=9.0",允许向前兼容的版本解析,涉及 runtimemetadef 及新增的依赖项。
  • 构建依赖补全:在 set_cann_build_dependencies 中新增 ge-executorasc-devkitopbase 三项依赖,版本约束均为 ">=9.0"
  • 运行依赖补全:在 set_cann_run_dependencies 中新增 ge-compilerge-executorasc-devkitopbase 四项依赖,版本约束均为 ">=9.0"
likedislike
sinat_31531339
sinat_31531339成员
6月30日 评论:

compile

likedislike
此处折叠了63条消息 查看更多
zhuliangying成员
7月1日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
7月1日 添加了label:lgtmapproved
CANN-robot
CANN-robot成员
7月1日 评论:

The MR can not be merged, because of CodeReview discussion not resolved

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike
sinat_31531339sinat_31531339成员
7月1日 解决了最后一个问题
CANN-robotCANN-robot成员
7月1日 合入了pull request