Skip to content
This repository has been archived by the owner on Oct 11, 2024. It is now read-only.

build: add djvu-imager-qt #2201

Merged
merged 1 commit into from
May 29, 2024
Merged

build: add djvu-imager-qt #2201

merged 1 commit into from
May 29, 2024

Conversation

wjyrich
Copy link
Contributor

@wjyrich wjyrich commented May 22, 2024

 It is based on the combination of the background glueing and mask coloring methods.

Log: add software name--djvu-imager-qt
djvu-imager-qt

IF(NOT WIN32 AND NOT APPLE)
- INSTALL(FILES res/djvu-imager-qt.png DESTINATION "/usr/share/pixmaps/")
- INSTALL(FILES debian/djvu-imager-qt.desktop DESTINATION "/usr/share/applications/")
+ INSTALL(FILES res/djvu-imager-qt.png DESTINATION share/icons/hicolor/)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

icon路径调整下,装到对应大小的目录下

 It is based on the combination of the background glueing and mask coloring methods.

Log: add software name--djvu-imager-qt
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • patches/0001-install.patch文件中,修改了安装脚本中的路径,需要确认新路径share/icons/hicolor/16X16/apps/是否正确,并且相应的系统环境变量和路径配置是否已经更新。
  • 检查CMakeLists.txt中的res/djvu-imager-qt.png文件是否存在于项目中,并且路径是否正确。
  • 确认share/icons/hicolor/16X16/apps/路径是否已经在构建系统或文档中进行说明和测试。

是否建议立即修改:

  • 是,需要确保路径配置的正确性和一致性,以避免在安装过程中出现路径错误。

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: kamiyadm, wjyrich

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

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

@kamiyadm kamiyadm merged commit c0b1357 into martyr-deepin:master May 29, 2024
6 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants