<table><tr><td style="">rjvbb accepted this revision.<br />rjvbb added a comment.<br />This revision is now accepted and ready to land.
</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/D11921">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">@rjvbb does this solve issues you are having and trying to workaround with https://phabricator.kde.org/D7930 ?</pre></div></blockquote>

<p>The problem I try to solve with <a href="https://phabricator.kde.org/D7930" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D7930</a> goes beyond having to rebuild everything every time: it is about code parsing failing when auto-generated (.moc) files use relative include paths. This can even fail during an actual build if the symlinked path doesn't have the same number of subdirectories relative to the root as the normalised path.</p>

<p>It looks like this patch should help with that issue, provided that the CMake stores the actual, normalised path in <tt style="background: #ebebeb; font-size: 13px;">CMAKE_CACHEFILE_DIR</tt>, does it?</p>

<p>I'm not noticing any different with and without this patch applied, on Mac. Not really surprising as the system always normalises paths, so applications cannot obtain a PWD path that contains symlinks.</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>I've never used CMAKE_CACHEFILE_DIR before and am not sure if it's OK to use it like this.</p></blockquote>

<p>According to the docs <tt style="background: #ebebeb; font-size: 13px;">This [variable] is the same as CMAKE_BINARY_DIR</tt> and thus should be the same path as KDevelop's $build.dir setting. The change should be safe so I agree with Sven: apply this to 5.2 but after the 5.2.2 release. Or better yet, delay that release a couple of days so more people can test and (many) more users can benefit from the fix a lot sooner.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11921">https://phabricator.kde.org/D11921</a></div></div><br /><div><strong>To: </strong>mwolff, KDevelop, rjvbb<br /><strong>Cc: </strong>kfunk, brauch, apol, kdevelop-devel<br /></div>