dschmidt updated this revision to Diff 36718. dschmidt added a comment.
Commit fixes to the actual ECMAddAppIconMacro REPOSITORY R240 Extra CMake Modules CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D13698?vs=36715&id=36718 BRANCH app-icon-modernization REVISION DETAIL https://phabricator.kde.org/D13698 AFFECTED FILES docs/find-module/FindIcoTool.rst find-modules/FindIcoTool.cmake modules/ECMAddAppIcon.cmake tests/ECMAddAppIconTest/128-category-name-sidebar.png tests/ECMAddAppIconTest/16-category-name-sidebar.png tests/ECMAddAppIconTest/CMakeLists.txt To: dschmidt, vonreth, vpinon, apol, alexmerry Cc: cgiboudeaux, kde-frameworks-devel, kde-buildsystem, michaelh, ngraham, bruns