Skip to content

Conversation

@dengbo11
Copy link
Contributor

Add mips64 linglong.yaml to support build linglong package.

Log: Support build mips64 linglong package.

Add mips64 linglong.yaml to support build linglong package.

Log: Support build mips64 linglong package.
@deepin-bot
Copy link
Contributor

deepin-bot bot commented Nov 6, 2025

TAG Bot

New tag: 6.5.41
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #199

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: dengbo11, lzwind

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@lzwind lzwind merged commit 465d361 into linuxdeepin:master Nov 13, 2025
4 of 5 checks passed
@github-actions
Copy link

CLA Assistant Lite bot:

如果你是以企业贡献者的身份进行提交,请联系我们签署企业贡献者许可协议
If you submit as corporate contributor, please contact us to sign our Corporate Contributor License Agreement

感谢您的提交,我们非常感谢。 像许多开源项目一样,在接受您的贡献之前,我们要求您签署我们的个人贡献者许可协议。 您只需发布与以下格式相同的评论即可签署个人贡献者许可协议
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Individual Contributor License Agreement before we can accept your contribution. You can sign the Individual Contributor License Agreement by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA.

You can retrigger this bot by commenting recheck in this Pull Request

@deepin-ci-robot
Copy link

deepin pr auto review

这是一个deepin-reader项目的linglong.yaml配置文件,我来从多个维度进行分析:

  1. 语法逻辑:
  • 配置文件整体结构完整,包含了必要的version、package、base、runtime、command、build和sources等字段。
  • build脚本部分逻辑清晰,包含了依赖安装、构建配置、编译和部署等步骤。
  • sources部分包含了大量依赖包,但格式统一。
  1. 代码质量:
  • 构建脚本中使用了sed命令修改配置文件,建议将这些修改集中到一个单独的脚本文件中,便于维护。
  • 部署文件生成部分使用了多个循环,可以考虑将相关操作封装成函数,提高代码可读性。
  • sources部分包含大量依赖包,建议添加注释说明各包的用途。
  1. 性能优化:
  • make命令使用了-jnproc参数,能够充分利用多核CPU进行并行编译。
  • 构建过程中使用了缓存目录build,避免重复编译。
  • 建议在构建过程中添加ccache支持,进一步提高编译速度。
  1. 安全性:
  • 所有依赖包都使用了digest进行完整性校验,确保包的安全性。
  • 建议在build脚本中添加错误处理机制,当某一步失败时能够及时终止构建过程。
  • 建议对下载的源码包进行安全扫描,特别是第三方依赖。
  1. 改进建议:
  • 添加构建环境变量配置部分,明确指定编译器和相关工具链版本。
  • 考虑添加构建测试步骤,确保构建产物质量。
  • 建议添加清理步骤,删除构建过程中产生的临时文件。
  • 可以考虑添加CI/CD配置,实现自动化构建和部署。
  • 建议添加详细的构建日志记录,便于问题追踪和调试。
  • 考虑添加多架构支持,方便在不同平台上构建。
  1. 其他建议:
  • 建议将版本号提取为变量,方便统一管理。
  • 可以考虑添加构建时间戳,便于追踪构建版本。
  • 建议添加构建环境要求说明,如最低内存、磁盘空间等。
  • 可以考虑添加构建产物大小限制,避免包体积过大。

这些改进建议可以帮助提高项目的可维护性、安全性和构建效率。

dengzhongyuan365-dev pushed a commit to dengzhongyuan365-dev/deepin-reader that referenced this pull request Nov 13, 2025
Add mips64 linglong.yaml to support build linglong package.

Log: Support build mips64 linglong package.

Co-authored-by: lzwind <100665065+lzwind@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants