<table><tr><td style="">asturmlechner 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/D24805">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D24805#550880" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D24805#550880</a>, <a href="https://phabricator.kde.org/p/hiangel/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@hiangel</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Do you have an example where having ccache on by default would cause a problem? When I asked what makes Konsole different from Mesa, libinput, etc, I was expecting to see an example.</p></div>
</blockquote>

<p>ccache is nothing new, you can imagine it will be a hot topic on a source-based distribution, so we have years of experience with it. Issues are mostly sandbox violations as are typically used for packaging builds, but also build errors, so much that on bug reports it is regularly demanded to rebuild without ccache.</p>

<blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D24805#550880" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D24805#550880</a>, <a href="https://phabricator.kde.org/p/hiangel/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@hiangel</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Btw, I just figured that even if ECM agrees all packages should use <tt style="background: #ebebeb; font-size: 13px;">ccache</tt> — what would happen then? I'm not sure that the task of adding <tt style="background: #ebebeb; font-size: 13px;">ccache</tt> autodetection to all existing projects is that automatable. It would be a lot of merge requests, and who's gonna address review? And what to do if some requests were accepted, and then someone addresses review to modify the changes: what to do with already accepted code…?</p></div>
</blockquote>

<p>No, it can be as simple as getting it approved into <tt style="background: #ebebeb; font-size: 13px;">KDECMakeSettings.cmake</tt> or <tt style="background: #ebebeb; font-size: 13px;">KDECompilerSettings.cmake</tt>, whatever fits best, and after upgrade to an ECM version with your change every project already using ECM will automatically have the option available. And enabling or disabling that stuff will be a matter for the person or script running make, not by cluttering dozens of individual CMakeLists.txt files.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24805">https://phabricator.kde.org/D24805</a></div></div><br /><div><strong>To: </strong>asturmlechner, Konsole<br /><strong>Cc: </strong>konsole-devel, hiangel, fbampaloukas, thsurrel, ngraham, maximilianocuria, hindenburg<br /></div>