D7865: Bring back per-project setting of the cmake executable completely

Morten Volden noreply at phabricator.kde.org
Mon Sep 18 16:13:18 UTC 2017


volden added inline comments.

INLINE COMMENTS

> cmakeutils.cpp:80
>  // TODO: migrate to more generic & consistent key term "CMake Executable"
> +static const QString cmakeBinaryKey = QStringLiteral("CMake Binary");
> +static const QString cmakeExecutableKey = QStringLiteral("CMake Executable");

Could you elaborate in the comment above why they are both here and when we would like to remove which variable.

REPOSITORY
  R32 KDevelop

REVISION DETAIL
  https://phabricator.kde.org/D7865

To: rjvbb, apol, volden
Cc: kdevelop-devel, #kdevelop, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170918/26f166e4/attachment-0001.html>


More information about the KDevelop-devel mailing list