Skip to content

Conversation

@re2zero
Copy link
Contributor

@re2zero re2zero commented Apr 9, 2025

The conflict between deepin-pdfium library and file manager installation, modify the name to avoid this issue.

Log: Rename pdf library.

The conflict between deepin-pdfium library and file manager installation, modify the name to avoid this issue.

Log: Rename pdf library.
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见:

  1. 文件路径和库文件名的一致性

    • src.pro文件中,目标名称从deepin-pdfium更改为deepin-pdfium-reader,这可能会导致构建系统中的其他部分没有相应地更新。需要检查所有引用该库的地方,确保路径和库名称的一致性。
  2. 构建脚本中的库文件名

    • arm64/linglong.yamllinglong.yamlloong64/linglong.yamlreader/src.pri文件中,库文件名从libdeepin-pdfium.so更改为libdeepin-pdfium-reader.so。这需要确保所有使用该库的代码都已经更新,并且库文件确实存在于指定的路径中。
  3. 构建脚本中的路径问题

    • src.proreader/src.pri文件中,路径使用了相对路径$$PWD/../3rdparty/deepin-pdfium。如果项目结构发生变化,这些路径可能会失效。建议使用绝对路径或者配置文件来管理这些路径。
  4. 代码质量

    • 检查是否有必要在所有相关文件中重复相同的更改。如果这些更改是必要的,考虑创建一个脚本来自动化这些更改,以减少重复劳动。
  5. 代码性能

    • 没有明显的性能问题,但建议定期审查代码,确保没有不必要的依赖或库加载。
  6. 代码安全

    • 检查是否有任何安全相关的更改,例如库版本更新是否引入了已知的安全漏洞。如果使用了第三方库,确保这些库是最新版本,并且没有已知的安全问题。

总结:

  • 确保所有引用库文件的地方都更新了库文件名和路径。
  • 考虑使用配置文件或脚本来自动化路径和库文件名的更改。
  • 定期审查代码,确保没有重复劳动,并且代码质量和性能得到维护。

@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 Apr 9, 2025

/merge

@deepin-bot deepin-bot bot merged commit adcfff6 into linuxdeepin:master Apr 9, 2025
6 checks passed
@re2zero re2zero deleted the bugfix branch April 9, 2025 08:49
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