Pull Request已成功合入, 合并人@ascend-robot
(感谢 hw-zhoutianyang 的贡献)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-Ascend/MindIE-LLM | ✅ 陈歆怡, earthmanylf (2/2) | ✅ 陈歆怡 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
hw-zhoutianyang, thanks for your pull request. All authors of the commits have signed the CLA. 👍


compile


Linking Issue Notice
@hw-zhoutianyang , the pull request must be linked to at least one issue.
If an issue has already been linked, but the needs-issue label remains, you can remove the label by commenting /check-issue .


✅ 流水线任务触发成功(流水线编号: 5067),正在执行, 请稍候。 任务链接 点此跳转 ...


问题/功能描述
本次PR对Makeself自解压脚本进行了重构与优化,主要解决了两方面问题:一是脚本内部文件权限管理逻辑分散且不够健壮;二是原有的X11图形终端自动检测与启动逻辑存在兼容性问题且增加了复杂度。重构旨在提升代码的清晰度、健壮性,并简化脚本在不同环境下的执行流程。
修改方案描述
修改方案包含两个核心部分:1. 将分散的文件权限设置代码重构为一个独立的change_permission函数,通过安全遍历和存在性检查,统一设置.run、.sh等文件的执行权限(550)和.info、.txt等文件的只读权限(440),并确保对临时目录的权限控制。2. 移除了基于DISPLAY环境变量和xset命令检测X11环境并自动启动xterm等终端模拟器的复杂逻辑,简化了脚本在非交互式或无图形界面环境下的执行路径,降低了潜在的兼容性风险。


| 阶段 | 任务名 | 状态 | 详情 |
|---|---|---|---|
| 编译构建 | Build_ops_x86_A3 | 🕚 | >>> |
| Build_ops_arm_A3 | 🕚 | >>> | |
| Build_ops_x86_A2 | 🕚 | >>> | |
| Build_ops_arm_A2 | 🕚 | >>> | |
| Build_linux_x86_abi1_atb | 🕚 | >>> | |
| Build_linux_x86_abi1_llm | 🕚 | >>> | |
| Build_linux_arm_abi1_atb | 🕚 | >>> | |
| Build_linux_arm_abi1_llm | 🕚 | >>> | |
| Build_linux_arm_abi0_atb | 🕚 | >>> | |
| Build_linux_arm_abi0_llm | 🕚 | >>> | |
| 恶意代码检查 | Antipoison | ✅ | >>> |
| 编码安全与规范检查 | pre-commit | ✅ | >>> |
| 开源片段检查 | SCA | ❌ | >>> |
| 开发者测试 | UT_linux_arm_mindie-llm | 🕚 | >>> |
| UT_linux_arm_atb-models | 🕚 | >>> | |
| 流水线 | PR-pipeline_MindIE-LLM_gitcode | ❌ | >>> |


Ascend docs pipeline is running...


Ascend docs pipeline is running...


✅ 跳过 docs ci 检查,没有需要检查的文档文件


✅ 跳过 docs ci 检查,没有需要检查的文档文件


compile


✅ 流水线任务触发成功(流水线编号: 5075),正在执行, 请稍候。 任务链接 点此跳转 ...


| 阶段 | 任务名 | 状态 | 详情 |
|---|---|---|---|
| 编译构建 | Build_ops_x86_A3 | ✅ | >>> |
| Build_ops_arm_A3 | ✅ | >>> | |
| Build_ops_x86_A2 | ✅ | >>> | |
| Build_ops_arm_A2 | ✅ | >>> | |
| Build_linux_x86_abi1_atb | ✅ | >>> | |
| Build_linux_x86_abi1_llm | ✅ | >>> | |
| Build_linux_arm_abi1_atb | ✅ | >>> | |
| Build_linux_arm_abi1_llm | ✅ | >>> | |
| Build_linux_arm_abi0_atb | ✅ | >>> | |
| Build_linux_arm_abi0_llm | ✅ | >>> | |
| 恶意代码检查 | Antipoison | ✅ | >>> |
| 编码安全与规范检查 | pre-commit | ✅ | >>> |
| 开源片段检查 | SCA | ✅ | >>> |
| 开发者测试 | UT_linux_arm_mindie-llm | ✅ | >>> |
| UT_linux_arm_atb-models | ✅ | >>> | |
| 流水线 | PR-pipeline_MindIE-LLM_gitcode | ✅ | >>> |


/lgtm


/approve


/merge


Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: 陈歆怡.
Reviewers who wrote a comment of /lgtm are: 陈歆怡, earthmanylf.


合入背景
Fixes #539
修改内容
修复mindie整包非root用户安装失败
资料变更
不涉及
接口变更
不涉及
测试结果
CheckList