<table><tr><td style="">bungeman created this revision.<br />bungeman added a reviewer: KDevelop.<br />Herald added a project: KDevelop.<br />Herald added a subscriber: kdevelop-devel.<br />bungeman requested review of this revision.
</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/D22197">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This sets the working directory of the clang parser similarly to the way<br />
the build directory is passed to the builder. This allows the parser to<br />
correctly resolve any relative paths in the extra build arguments. This<br />
means relative paths are resolved relative to the build directory (as<br />
they would be resolved by a builder).</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>In the current test setup it doesn't seem the TestProject allows a TestBuildSystemManager, so it seems like quite a bit of work to create a test for this. (As a result I assume IBuildSystemManager integration isn't well tested in general right now.) If this needs a test, let me know if adding a TestBuildSystemManager is the right way to go.</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/D22197">https://phabricator.kde.org/D22197</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>plugins/clang/clangparsejob.cpp<br />
plugins/clang/duchain/clangparsingenvironment.cpp<br />
plugins/clang/duchain/clangparsingenvironment.h<br />
plugins/clang/duchain/parsesession.cpp</div></div></div><br /><div><strong>To: </strong>bungeman, KDevelop<br /><strong>Cc: </strong>kdevelop-devel, hmitonneau, christiant, glebaccon, domson, antismap, iodelay, alexeymin, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd<br /></div>