<table><tr><td style="">akellermann updated this revision to Diff 18068.<br />akellermann 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/D7040" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I had access to a windows machine running kdevelop and had a look at the format of cmake output there.  So this patch</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">updates the regex to work with cmake project paths on Windows (format is D:/path/to/proj)</li>
<li class="remarkup-list-item">adds unit tests for Linux paths with and without spaces</li>
<li class="remarkup-list-item">adds unit tests for Windows paths with and without spaces</li>
</ol>

<p>Unfortunately the Windows tests don't work at the moment (and are therefore commented out), as class Path used in CompilerFilterStrategyPrivate to store paths mangles Windows paths starting with drive letters (transforms e.g. <tt style="background: #ebebeb; font-size: 13px;">D:/</tt> to <tt style="background: #ebebeb; font-size: 13px;">d:///</tt>). I'm not sure how to solve this, or if this should be solved with this patch at all, as it blows the scope out of proportion.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D7040?vs=18010&id=18068" rel="noreferrer">https://phabricator.kde.org/D7040?vs=18010&id=18068</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7040" rel="noreferrer">https://phabricator.kde.org/D7040</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>outputview/outputfilteringstrategies.cpp<br />
outputview/tests/test_filteringstrategy.cpp<br />
outputview/tests/testlinebuilderfunctions.h</div></div></div><br /><div><strong>To: </strong>akellermann, kfunk<br /><strong>Cc: </strong>volden, kdevelop-devel, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger<br /></div>