daandemeyer created this revision.
daandemeyer added reviewers: alexmerry, dschmidt.
daandemeyer added a project: Frameworks.
Herald added subscribers: kde-buildsystem, kde-frameworks-devel.
Herald added a project: Build System.
daandemeyer requested review of this revision.

REVISION SUMMARY
  Same as https://phabricator.kde.org/D22698 for `ecm_add_app_icon`.

TEST PLAN
  I don't have KDE setup on Windows or access to a Mac and I think this code is 
only fully exercised on those system. I confirmed it doesn't break anything on 
Linux by running kate's CMake configuration but that's only half of the work 
since most of the macro isn't used on Linux. If anyone could verify this patch 
works on Windows or Mac it would be much appreciated.

REPOSITORY
  R240 Extra CMake Modules

REVISION DETAIL
  https://phabricator.kde.org/D22709

AFFECTED FILES
  modules/ECMAddAppIcon.cmake

To: daandemeyer, alexmerry, dschmidt
Cc: kde-frameworks-devel, kde-buildsystem, LeGast00n, sbergeron, bencreasy, 
michaelh, ngraham, bruns

Reply via email to