<table><tr><td style="">mwolff 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/D9297" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>can you turn the test projects into proper unit tests that can be run in an automated fashion to see what happens here?</p>

<p>also the <tt style="background: #ebebeb; font-size: 13px;">absoluteFilePath</tt> call in kdev-clang is only used for the (undocumented) <tt style="background: #ebebeb; font-size: 13px;">.kdev_pch_include</tt> feature - so that is unrelated. It should indeed probably use the canonical file path though. Anyhow, since this is clearly unrelated to what you are trying to fix, I wonder if your change actually helps? Is it maybe the call to <tt style="background: #ebebeb; font-size: 13px;">absoluteFilePath</tt> in <tt style="background: #ebebeb; font-size: 13px;">plugins/custom-definesandincludes/noprojectincludesanddefines/noprojectincludesanddefinesmanager.cpp</tt>'s <tt style="background: #ebebeb; font-size: 13px;">NoProjectIncludePathsManager::findConfigurationFile</tt> that should be turned into a canonical path?</p>

<p>In general, it might be a good idea to go through our code base and replace all calls of <tt style="background: #ebebeb; font-size: 13px;">absoluteFilePath</tt> with their canonical equivalents... But let's not do that blindly, rather on a case-by-case study.</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/D9297" rel="noreferrer">https://phabricator.kde.org/D9297</a></div></div><br /><div><strong>To: </strong>vbspam, KDevelop<br /><strong>Cc: </strong>mwolff, brauch, kdevelop-devel, iodelay, vbspam, njensen, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger<br /></div>