<table><tr><td style="">rjvbb created this revision.<br />rjvbb added reviewers: KDevelop, apol, volden.<br />rjvbb added a project: KDevelop.<br />Restricted Application added a subscriber: kdevelop-devel.
</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/D7887" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This cleans up the duplicate methods and operations that resulted from the recent merge of Morten's cmake executable preferences patch to the main branch with my own to the 5.2 branch.</p>

<p>This patch was made against the master branch.</p>

<p>I've standardised on <tt style="background: #ebebeb; font-size: 13px;">currentCMakeExecutable()</tt> for consistency with existing conventions and stuck to my approach that centralises the fallback from project-setting to kdevelop-setting to hardcoded system default.</p>

<p>I've kept Morten's simpler use of a QString& argument in the setter, though.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>builds and runs as intended (tested on the 5.2 branch).</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/D7887" rel="noreferrer">https://phabricator.kde.org/D7887</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>plugins/cmake/cmakeutils.cpp<br />
plugins/cmake/cmakeutils.h<br />
plugins/cmake/settings/cmakepreferences.cpp<br />
plugins/cmakebuilder/cmakejob.cpp</div></div></div><br /><div><strong>To: </strong>rjvbb, KDevelop, apol, volden<br /><strong>Cc: </strong>kdevelop-devel, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger<br /></div>