已合并
[build] unified encoding, line breaks, and license header format #4522
ouyangcaike创建于 8月10日
[build] unified encoding, line breaks, and license header format #4522
已合并
Pull Request已成功合入, 合并人@CANN-robot
（感谢 ouyangcaike 的贡献）8月10日 创建了 pull request，commit 8686518b
atomgit-bot
8月10日 评论：
8月10日 评论：
变更摘要
此 PR 是一个纯工程性变更，主要将 rdma_agent 和 rdma_service 相关构建模块文件以及拓扑信息分区源文件的编码格式从 GB-2312 转换为 UTF-8，同时为 5 个 module.mk 文件补充了 CANN Open Software License Agreement Version 2.0 版权声明头部。所有变更均为编码层面的转换，不涉及任何功能逻辑、接口或配置的修改。
主要改动
- 为
module.mk文件添加版权声明头部：5 个module.mk文件（adapter、client、hdc、peer、rdma_service）均新增了 10 行 CANN Open Software License Agreement Version 2.0 的许可证头部注释。 - 将
module.mk中的中文注释转为 UTF-8 编码：上述 5 个文件中原有的 GB-2312 编码的中文注释（如# LOCAL_PATH用于build核心加上LOCAL_SRC_FILES组合成源文件、#第三方头文件搜索路径、#第三方库搜索路径等）统一转换为 UTF-8 编码。 - 将
topoinfo_ranktable_partition.cc中的中文注释转为 UTF-8 编码：TopoinfoRanktablePartition::GenerateSubSuperPodId函数内的 8 处中文注释（如逻辑超节点范围记录、rankId 连续性判断、范围初始化与更新等说明）从 GB-2312 编码转换为 UTF-8 编码，代码逻辑本身无变化。
不准确？
atomgit-bot
8月10日 评论：
8月10日 评论：
8月10日 添加了label：cann-cla/yes
CANN-robot
8月10日 评论：
8月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
✅ Congratulations! All modules have met the lgtm and approve requirements.
Module Approval Details
| module | lgtm status | approve status |
|---|---|---|
| .gitcode/** | ✅ 宋常霞, 颜业峰, 劳大钊 (3/2) | ✅ 宋常霞 (1/1) |
| classify_rule.yaml | ✅ 劳大钊, 颜业峰 (2/2) | ✅ 颜业峰 (1/1) |
| cmake | ✅ 颜业峰, 劳大钊 (2/2) | ✅ 颜业峰 (1/1) |
| docs | ✅ 颜业峰, 劳大钊 (2/2) | ✅ 颜业峰 (1/1) |
| examples | ✅ 颜业峰, 劳大钊 (2/2) | ✅ 颜业峰 (1/1) |
| experimental | ✅ 颜业峰, 劳大钊 (2/2) | ✅ 颜业峰 (1/1) |
| repo-cann/hcomm | ✅ 颜业峰, 劳大钊 (2/2) | ✅ 颜业峰 (1/1) |
| scripts | ✅ 颜业峰, 劳大钊 (2/2) | ✅ 颜业峰 (1/1) |
| src/base_comm | ✅ 颜业峰, 劳大钊 (2/2) | ✅ 颜业峰 (1/1) |
| src/base_comm/resources/hccp | ✅ 颜业峰, 劳大钊 (2/2) | ✅ 颜业峰 (1/1) |
| src/coll_communicator_mgr | ✅ 劳大钊, 颜业峰 (2/2) | ✅ 颜业峰 (1/1) |
| src/legacy/ascend910/framework | ✅ 颜业峰, 劳大钊 (2/2) | ✅ 颜业峰 (1/1) |
| src/legacy/ascend950/framework | ✅ 颜业峰, 劳大钊 (2/2) | ✅ 颜业峰 (1/1) |
| test | ✅ 颜业峰, 劳大钊 (2/2) | ✅ 颜业峰 (1/1) |
| test/hccl_vm | ✅ 詹海峰, 颜业峰, 劳大钊 (3/2) | ✅ 詹海峰 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
ouyangcaike, thanks for your pull request. All authors of the commits have signed the CLA. 👍
此处折叠了357条消息 查看更多
15 天前 添加了label：lgtm
songchangxia
15 天前 评论：
15 天前 评论：
/lgtm
/approve
15 天前 添加了label：approved
15 天前 合入了pull request
描述
变更类型
请选择本次引入的变更类型：
关联的Issue
NA
测试
已完成的测试用例和场景：
补充的UT用例：
NA
文档更新
NA
合入检查