<table><tr><td style="">daandemeyer created this revision.<br />daandemeyer added reviewers: alexmerry, dschmidt.<br />daandemeyer added a project: Frameworks.<br />Herald added subscribers: kde-buildsystem, kde-frameworks-devel.<br />Herald added a project: Build System.<br />daandemeyer requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D22709">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Same as <a href="https://phabricator.kde.org/D22698" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D22698</a> for <tt style="background: #ebebeb; font-size: 13px;">ecm_add_app_icon</tt>.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>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.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R240 Extra CMake Modules</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22709">https://phabricator.kde.org/D22709</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>modules/ECMAddAppIcon.cmake</div></div></div><br /><div><strong>To: </strong>daandemeyer, alexmerry, dschmidt<br /><strong>Cc: </strong>kde-frameworks-devel, kde-buildsystem, LeGast00n, sbergeron, bencreasy, michaelh, ngraham, bruns<br /></div>