----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124595/#review84022 -----------------------------------------------------------
Ship it! Very extensive, many many thanks! modules/ECMQtDeclareLoggingCategory.cmake (line 33) <https://git.reviewboard.kde.org/r/124595/#comment58203> 5.14 now - David Faure On Aug. 16, 2015, 9:34 p.m., Alex Merry wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124595/ > ----------------------------------------------------------- > > (Updated Aug. 16, 2015, 9:34 p.m.) > > > Review request for Extra Cmake Modules and David Faure. > > > Repository: extra-cmake-modules > > > Description > ------- > > This makes life a bit easier for developers who use the categorised > logging in Qt5 in the common case - rather than creating two new files, > and remembering to put in the #ifdef for the default verbosity settings > in Qt 5.4, they can just add a couple of lines to their CMakeLists.txt. > > (NB: see https://codereview.qt-project.org/#/c/122641 for patch for the Qt > bug mentioned in the docs). > > > Diffs > ----- > > tests/ECMQtDeclareLoggingCategoryTest/testmain.cpp PRE-CREATION > tests/ECMQtDeclareLoggingCategoryTest/CMakeLists.txt PRE-CREATION > tests/CMakeLists.txt 7960154a42ec083be61aec2bd066b072560beb61 > modules/ECMQtDeclareLoggingCategory.h.in PRE-CREATION > modules/ECMQtDeclareLoggingCategory.cpp.in PRE-CREATION > docs/module/ECMQtDeclareLoggingCategory.rst PRE-CREATION > modules/ECMQtDeclareLoggingCategory.cmake PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/124595/diff/ > > > Testing > ------- > > Added unit test runs and passes. Documentation builds and looks sane. > > > Thanks, > > Alex Merry > >
_______________________________________________ Kde-buildsystem mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-buildsystem
