Review Request 127432: ecm_qt_declare_logging_category: improve error message when using without including

David Faure faure at kde.org
Sun Mar 20 10:53:52 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127432/
-----------------------------------------------------------

Review request for Extra Cmake Modules and Stephen Kelly.


Repository: extra-cmake-modules


Description
-------

If one subdir in the project includes this file, all others can use the function
but they don't see the value of the variable, which leads to a strange error

CMake Error at ECM/modules/ECMQtDeclareLoggingCategory.cmake:114 (configure_file):
  configure_file input location <srcdir> is a directory but a file was expected.

Happened in KIO, with kio/gui doing include+function call, and then adding
function call in kio/widgets.


Diffs
-----

  modules/ECMQtDeclareLoggingCategory.cmake 3f7bb79a7f4d98c1480f87b1fffc252f4e159add 

Diff: https://git.reviewboard.kde.org/r/127432/diff/


Testing
-------

As described in above. Got the better error message after the fix.


Thanks,

David Faure

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20160320/07911bba/attachment.html>


More information about the Kde-buildsystem mailing list