<table><tr><td style="">kossebau 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/D5632" rel="noreferrer">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/D5632#105488" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;" rel="noreferrer">D5632#105488</a>, <a href="https://phabricator.kde.org/p/kfunk/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@kfunk</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Uh, careful here. This suggests those binaries are *required* at runtime.</p>

<p>From <a href="https://cmake.org/cmake/help/v3.0/module/FeatureSummary.html" class="remarkup-link" target="_blank" rel="noreferrer">https://cmake.org/cmake/help/v3.0/module/FeatureSummary.html</a>:</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Last, a RUNTIME package is a package which is actually not used at all during the build, but which is required for actually running the resulting binaries.</p></blockquote>

<p>Please make double-sure packagers are aware this is only a *suggestion*, not required at runtime for KDevelop to function properly. Maybe rephrase the <tt style="background: #ebebeb; font-size: 13px;">PURPOSE</tt> field:<br />
 <tt style="background: #ebebeb; font-size: 13px;">PURPOSE "Suggestion: Required for the heaptrack plugin"</tt>?</p></div>
</blockquote>

<p>Okay, will do a fix-up commit.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>TBH, I'm not sure we need this patch at all.</p></blockquote>

<p>One thing that needs improvement currently is that the heaptrack and cppcheck plugins are active by default, and inject their actions into the UI enabled even if there is no cppcheck or heaptrack installed at all. Which was my starting motivation for this patch.<br />
Still if optional I think noting the optional runtime deps in cmake summary result is a good way to make packagers and other people building it aware of these dependencies.</p>

<p>CMake should rather be improved to make RUNTIME something orthogonal to OPTIONAL/RECOMMENDED/REQUIRED.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5632" rel="noreferrer">https://phabricator.kde.org/D5632</a></div></div><br /><div><strong>To: </strong>kossebau, KDevelop, apol, kfunk<br /><strong>Cc: </strong>kfunk, kdevelop-devel<br /></div>