Review Request 122135: Add ecm_add_app_icon function.
Alex Merry
alex.merry at kde.org
Sun Jan 18 22:32:45 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122135/
-----------------------------------------------------------
Review request for Extra Cmake Modules, Ralf Habacker and Jeremy Whiting.
Repository: extra-cmake-modules
Description
-------
This adds an application icon to an executable from PNG files for
Windows and Mac OS X. Unlike the similar kde4_add_app_icon macro from
kdelibs, this requires icons to be explicitly listed as arguments
(meaning CMake can tell when ones are added or deleted and reconfigure
as appropriate), and it works with Matthias Benkmann's png2ico tool, as
well as the KDE-Win tool of the same name.
Currently missing unit tests. Also completely untested (except that
`make test` runs on Linux, so there are no obvious syntax errors).
CHANGELOG: Add ECMAddAppIcon module to add icons to executable targets
on Windows and Mac OS X.
Diffs
-----
docs/find-module/FindPng2Ico.rst PRE-CREATION
docs/module/ECMAddAppIcon.rst PRE-CREATION
find-modules/FindPng2Ico.cmake PRE-CREATION
modules/ECMAddAppIcon.cmake PRE-CREATION
Diff: https://git.reviewboard.kde.org/r/122135/diff/
Testing
-------
`make test` passes, which just provides a very basic check for syntax errors.
Thanks,
Alex Merry
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20150118/ed40ac94/attachment.html>
More information about the Kde-buildsystem
mailing list