D9446: WIP: Allow to autogenerate and install categories file
Ralf Habacker
noreply at phabricator.kde.org
Thu Jun 28 08:53:39 UTC 2018
habacker added a comment.
- doc in file header is missing
- test case is missing
INLINE COMMENTS
> ECMQtDeclareLoggingCategory.cmake:68
> set(options)
> - set(oneValueArgs HEADER IDENTIFIER CATEGORY_NAME DEFAULT_SEVERITY)
> + set(oneValueArgs HEADER IDENTIFIER CATEGORY_NAME DEFAULT_SEVERITY CATEGORY_INSTALL_FILENAME CATEGORY_DESCRIPTION)
> set(multiValueArgs)
may be better using shorter names like INSTALL_FILENAME, DESTINATION or FILENAME
and DESCRIPTION instead of CATEGORY_DESCRIPTION. because we are declaring a logging category
> ECMQtDeclareLoggingCategory.cmake:138
> + if (ARG_CATEGORY_INSTALL_FILENAME)
> + set(CAT_DESCRIPTION)
> + if (ARG_CATEGORY_DESCRIPTION)
Using ${PROJECT_NAME} if empty ?
REPOSITORY
R240 Extra CMake Modules
REVISION DETAIL
https://phabricator.kde.org/D9446
To: mlaurent, kfunk, lbeltrame, cgiboudeaux, dfaure, fvogt
Cc: ltoscano, kde-frameworks-devel, kde-buildsystem, habacker, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20180628/cce898d0/attachment.html>
More information about the Kde-buildsystem
mailing list