Skip to content

Conversation

@re2zero
Copy link
Contributor

@re2zero re2zero commented Mar 27, 2025

Fix CI and remove unused dependency packages

Log: Fix CI and remove unused dependency package.

Fix CI and remove unused dependency packages

Log: Fix CI and remove unused dependency package.
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见:

  1. .gitignore 文件:

    • 新增的 .vscode 目录应该有注释说明为什么需要忽略这个目录,以便其他开发者理解。
  2. debian/control 文件:

    • 移除了 libspectre-dev 依赖,需要确认是否有其他地方使用了 libspectre,如果没有,这个改动是合理的。如果有,应该重新添加这个依赖。
  3. debian/rules 文件:

    • rules 文件中添加了 export QT_SELECT=5,这个改动是为了兼容某些需要 Qt5 环境的构建。但是,这个环境变量应该只在需要的时候设置,而不是全局设置。可以考虑只在构建脚本中设置这个环境变量。
  4. deepin_reader.pro 文件:

    • 移除了 poppler-qt5 依赖,需要确认是否有其他地方使用了 poppler-qt5,如果没有,这个改动是合理的。如果有,应该重新添加这个依赖。
  5. reader/reader.pro 文件:

    • 同样移除了 poppler-qt5 依赖,需要确认是否有其他地方使用了 poppler-qt5,如果没有,这个改动是合理的。如果有,应该重新添加这个依赖。
  6. reader/src.pri 文件:

    • 同样移除了 poppler-qt5 依赖,需要确认是否有其他地方使用了 poppler-qt5,如果没有,这个改动是合理的。如果有,应该重新添加这个依赖。

总结:

  • 所有移除 poppler-qt5 依赖的地方都需要确认是否有其他地方使用了这个库,如果没有,这个改动是合理的。
  • debian/rules 文件中设置 QT_SELECT 环境变量应该只在需要的时候设置,而不是全局设置。
  • .gitignore 文件中新增的 .vscode 目录应该有注释说明为什么需要忽略这个目录。

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

@re2zero
Copy link
Contributor Author

re2zero commented Mar 27, 2025

/merge

@deepin-bot deepin-bot bot merged commit c43e274 into linuxdeepin:master Mar 27, 2025
6 checks passed
@re2zero re2zero deleted the bugfix branch March 27, 2025 03:37
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