Review Request 127432: ecm_qt_declare_logging_category: improve error message when using without including
Stephen Kelly
steveire at gmail.com
Sun Mar 20 18:42:07 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127432/#review93800
-----------------------------------------------------------
Ship it!
Ship It!
- Stephen Kelly
On March 20, 2016, 10:53 a.m., David Faure wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127432/
> -----------------------------------------------------------
>
> (Updated March 20, 2016, 10:53 a.m.)
>
>
> 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/bc7e7c1d/attachment.html>
More information about the Kde-buildsystem
mailing list