<table><tr><td style="">patrickelectric added a comment.
</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/D26752">View Revision</a></tr></table><br /><div><div><p>Hi <a href="https://phabricator.kde.org/p/cgiboudeaux/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@cgiboudeaux</a> and <a href="https://phabricator.kde.org/p/bcooksley/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@bcooksley</a>, there is a reason of why this patch is valid. Have you read the commit message ?</p>

<p>In <a href="https://phabricator.kde.org/tag/kirogi/" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">#kirogi</a> we provide a valid icon (svg) with a valid prefix (sc), as you probably know *sc* stands for for scalable (SVG) files.</p>

<p>As I said in the commit body, "KSVG2ICNS will not exist if the program is not being compiled to APPLE"<br />
You'll probably say: "Well, so the developer should install this program."<br />
Nops, if you take a look in <strong>kiconthemes</strong>: <a href="https://github.com/KDE/kiconthemes/blob/3e668c7fba9fda7469a75247e0926530cdd1eb29/src/CMakeLists.txt#L3" class="remarkup-link" target="_blank" rel="noreferrer">https://github.com/KDE/kiconthemes/blob/3e668c7fba9fda7469a75247e0926530cdd1eb29/src/CMakeLists.txt#L3</a><br />
You'll see that KDE only provides such binary if APPLE is true.<br />
Why such binary is important ? As you can see in <strong>ECMAddAppIcon</strong>: <a href="https://github.com/KDE/extra-cmake-modules/blob/master/modules/ECMAddAppIcon.cmake#L117" class="remarkup-link" target="_blank" rel="noreferrer">https://github.com/KDE/extra-cmake-modules/blob/master/modules/ECMAddAppIcon.cmake#L117</a><br />
There is a <strong>APPLE</strong> check to convert the SVG binaries to mac valid icons.<br />
Well, if <strong>KSVG2ICNS</strong> is only build to APPLE.<br />
<strong>ECMAddAppIcon</strong> only looks for <strong>KSVG2ICNS</strong> with <strong>APPLE</strong>.<br />
Why should <strong>ECMAddAppIcon</strong> provide such warning if KDE <strong>ECM</strong> and<strong>kiconthemes</strong> only do such thing for APPLE ?<br />
There is no logic reason for this warning if everything else only works with <strong>APPLE</strong>.<br />
If the developer wants to cross build for <strong>APPLE</strong> he should set <strong>APPLE</strong> and build everything to <strong>APPLE</strong>, this warning is still not valid with the logic present in this comment. <br />
I did point using the code and the workflow of the ECM files why this patch is valid, please provide the same thing if not.</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/D26752">https://phabricator.kde.org/D26752</a></div></div><br /><div><strong>To: </strong>patrickelectric, apol, tcanabrava, cgiboudeaux, bcooksley<br /><strong>Cc: </strong>bcooksley, patrickelectric, apol, cgiboudeaux, kde-frameworks-devel, kde-buildsystem, LeGast00n, GB_2, bencreasy, michaelh, ngraham, bruns<br /></div>