D22699: Support passing target to ecm_qt_declare_logging_category
Christophe Giboudeaux
noreply at phabricator.kde.org
Wed Jul 24 12:35:26 BST 2019
cgiboudeaux added inline comments.
INLINE COMMENTS
> daandemeyer wrote in ECMQtDeclareLoggingCategory.cmake:133
> I found that the CMake script of extra-cmake-modules itself specifies 2.8.12 as the minimum so I'm assuming that's the minimum version for the modules as well. The advantage of specifying cmake_minimum_required inline here is that cmake-extra-modules will continue to work with versions of CMake older than 3.1 and give a better error message when the new functionality is used with an older CMake version. Of course, if its not a problem to update the minimum required version to 3.1 that's the preferred solution compared to specifying cmake_minimum_required inline.
If the change cannot be made compatible with the minimum ECM version, I suggest waiting for KF6.
REPOSITORY
R240 Extra CMake Modules
REVISION DETAIL
https://phabricator.kde.org/D22699
To: daandemeyer, alexmerry
Cc: cgiboudeaux, apol, kde-frameworks-devel, kde-buildsystem, LeGast00n, sbergeron, bencreasy, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20190724/55320cd3/attachment.html>
More information about the Kde-buildsystem
mailing list