已合并
dflow、ge-compiler、ge-executor安装权限规范统一 #4546
qinhan创建于 24 天前
dflow、ge-compiler、ge-executor安装权限规范统一 #4546
已合并
qinhan创建于 24 天前
qinhan
qinhan
24 天前

Pull Request

描述

统一dflow、ge-compiler、ge-executor安装文件及目录权限,这些权限统一由cmake控制, 删除安装脚本中一些影响权限规范的操作

变更类型

请选择本次引入的变更类型:

关联的Issue

如何测试

描述测试此变更的步骤和前提条件:
重新出包,然后比较安装后的内容化是否符合预期
除了opp/built-in目录为550,所有目录均为750,可执行程序为750,python/shell脚本为750,
share/info/包名/script目录下安装卸载脚本等统一为740,其他文件为640,符号链接为777

核对清单

其他信息

在此添加任何其他关于本次 PR 的说明。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 qinhan 的贡献)
qinhanqinhan
24 天前 添加了label:enhancement
qinhanqinhan
24 天前 创建了 pull request,commit fc1105cb
atomgit-bot
atomgit-bot
24 天前 评论:

变更摘要

本 PR 旨在统一 dflowge-compilerge-executor 三个组件安装文件及目录的权限规范。改动方式为纯删除(无新增代码):移除散落在各安装脚本与功能脚本中的 chmod 权限设置,包括数据文件、version.info、Python 包目录、stub 软链接目录等,使目录/文件权限不再由各脚本单独指定,统一交由统一规范管控,脚本中仅保留 chown 属主设置等必要操作。

主要改动

  • 移除安装目录与数据文件权限设置scripts/package/dflow-executor/scripts/install.shscripts/package/ge-executor/scripts/install.shscripts/package/ge-compiler/scripts/install.shchmod_end() 中删除了对 ascend_install.infoversion.infoscene.infodefault_dirscript 目录/文件的 chmod 调用,仅保留 chown -R 属主设置。
  • 移除 ge-compiler 的 Python 包权限设置scripts/package/ge-compiler/scripts/install.shchmod_end() 中删除了对 pythonsite-packagesllm_datadist_v1dataflowgege_pyge_py_pass_bridge 等目录和 dist-info、LICENSE 的大量递归 chmod 设置。
  • 移除版本信息文件权限设置:三个组件的 install.shupdate_version_info_version() 均不再对 $default_dir/version.info 执行 chmod_single_dir ... 440
  • 移除 stub 软链接目录权限操作scripts/package/ge-compiler/scripts/ge-compiler_func.shscripts/package/ge-executor/scripts/ge-executor_func.shcreate_stub_softlink() 中移除 chmod u-w .remove_stub_softlink() 中移除操作结束后的 chmod u-w .(保留前置的 chmod u+w .)。
  • 移除 ACL 头文件目录只读设置scripts/package/ge-executor/scripts/ge-executor_func.shcreate_acl_empty_headers() 中删除了对 $acl_headers_dirchmod u-w 调用。
likedislike
不准确?
atomgit-bot
atomgit-bot
24 天前 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robot
CANN-robot成员
24 天前 评论:

Hi @qinhan123, welcome to submitting your first PR to ge!

PR Merge Steps

1. CLA Signing

If the current PR label includes cann-cla/yes, it means you have signed the CLA and can proceed to the next step. If the label includes cann-cla/no, please sign the CLA first. If you have any questions, please refer to the FAQ.

2. CI Check

Please comment /compile to trigger the CI pipeline check. If the CI run is successful, the PR will be tagged with ci-pipeline-passed and you can proceed to the next step. If the CI run fails, the PR will be tagged with ci-pipeline-failed, please check the CI logs to fix the issues in the PR. If you have any questions, please refer to the FAQ.

3. Code Review

After CI passes, please refer to the PR Approval Progress and proactively @ the committers in the table to review the code. After approval, committers will comment /lgtm and /approve. Once the lgtm and approved labels are successfully added, the PR will be merged automatically.

likedislike
CANN-robotCANN-robot成员
24 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
24 天前 评论:

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/ge yangyongqiang0606, kobemini (2/2) yangyongqiang0606 (1/1)

💡 Tip:

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

CLA Signature Pass

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

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
24 天前 将yangyongqiang0606,kobemini,ClarkXie,tangqunzhang,xchu42,hugo111,lining23666,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为评审人
此处折叠了6条事件消息 查看更多
CANN-robotCANN-robot成员
24 天前 添加了label:cann-cla/yes
qinhan
qinhan
24 天前 评论:

compile

likedislike
qinhanqinhan
24 天前 预合并成功(commit_id: 78b1e2f19ee681ab973188c0df21837139f00d14)
qinhanqinhan
24 天前 修改了pull request 的描述
CANN-robotCANN-robot成员
24 天前 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
24 天前 评论:

流水线任务触发成功
任务链接 [b095be6e07de4a689c4192139675b9ad][流水线指导]

任务名称状态日志下载链接
pre_comment ✅ SUCCESS >>>>>
Compile_ARM_compiler ✅ SUCCESS >>>>> >>>>>
Compile_ARM_executor ✅ SUCCESS >>>>> >>>>>
Compile_ARM_dflow ✅ SUCCESS >>>>> >>>>>
API_Check ✅ SUCCESS >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>

[2026-08-25 20:39:57]    CI执行结束

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

流水线任务触发成功
任务链接 [203c17e6e57e49ec8ba40ff0262fe603][流水线指导]

任务名称状态日志下载链接
pre_comment ✅ SUCCESS >>>>>
Compile_X86_compiler ✅ SUCCESS >>>>> >>>>>
Compile_X86_executor ✅ SUCCESS >>>>> >>>>>
Compile_X86_dflow ✅ SUCCESS >>>>> >>>>>
UT_Test_dflow ✅ SUCCESS
UT_Test_ge_common ✅ SUCCESS
UT_Test_parser ✅ SUCCESS
UT_Test_python ✅ SUCCESS >>>>>
UT_Test_rt ✅ SUCCESS
UT_Test_acl ✅ SUCCESS
UT_Test_autofuse ✅ SUCCESS
UT_Test_executor ✅ SUCCESS
UT_Test_autofuse_ascendc_api ✅ SUCCESS
ST_Test_dflow ✅ SUCCESS
ST_Test_ge_common ✅ SUCCESS
ST_Test_parser ✅ SUCCESS
ST_Test_python ✅ SUCCESS >>>>>
ST_Test_rt ✅ SUCCESS
ST_Test_autofuse ✅ SUCCESS
ST_Test_executor ✅ SUCCESS
ST_Test_hetero ✅ SUCCESS
ST_Test_autofuse_ascendc_api ✅ SUCCESS
ST_Test_autofuse_e2e ✅ SUCCESS
UT_Test_fe ✅ SUCCESS
UT_Test_tefusion ✅ SUCCESS
ST_Test_fe ✅ SUCCESS
ST_Test_tefusion ✅ SUCCESS
ST_Test_ge_common_atc ✅ SUCCESS
UT_Test_Report_graphengine ✅ SUCCESS >>>>>
UT_Test_Report_ge_executor_c ✅ SUCCESS >>>>>
UT_Test_Report_ge_autofuse ✅ SUCCESS >>>>>
ST_Test_Report_graphengine ✅ SUCCESS >>>>>
ST_Test_Report_ge_executor_c ✅ SUCCESS >>>>>
ST_Test_Report_ge_autofuse ✅ SUCCESS >>>>>
UT_Test_Report_fe ✅ SUCCESS >>>>>
ST_Test_Report_fe ✅ SUCCESS >>>>>

[2026-08-25 20:47:47]    CI执行结束

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

流水线任务触发成功
任务链接 [7480a00472a7477d836bd06d6af2b6d8][流水线指导]

任务名称状态日志下载链接
codecheck_Pr ✅ SUCCESS
antipoison ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
codecheck_dt ✅ SUCCESS >>>>>
codecheck_precommit ✅ SUCCESS >>>>>
StaticCheck_codespell ✅ SUCCESS
StaticCheck_link_validity ✅ SUCCESS
StaticCheck_resource_existence ✅ SUCCESS
StaticCheck_tag_closed ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS

[2026-08-25 20:38:06]    CI执行结束

likedislike
CANN-robotCANN-robot成员
24 天前 添加了label:api-check-pass
此处折叠了7条事件消息 查看更多
CANN-robotCANN-robot成员
17 天前 删除了label:ci-pipeline-passed
CANN-robot
CANN-robot成员
17 天前 评论:

Notification

This pull request has been changed(code update) or closed, so removes the following label(s): ci-pipeline-passed.

likedislike
CANN-robotCANN-robot成员
17 天前 添加了label:cann-cla/yes
qinhan
qinhan
17 天前 评论:

compile

likedislike
qinhanqinhan
17 天前 预合并成功(commit_id: 681b4e17fcb11c3e4b44bd80fb8c8756b83ecfb7)
此处折叠了5条事件消息 查看更多
qinhanqinhan
17 天前 修改了pull request 的描述
GengChao
GengChao成员
17 天前 评论:

/lgtm

likedislike
yangyongqiang
yangyongqiang成员
17 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
17 天前 添加了label:lgtm
yangyongqiang
yangyongqiang成员
16 天前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
16 天前 添加了label:approved
CANN-robotCANN-robot成员
16 天前 合入了pull request