----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115586/#review49343 -----------------------------------------------------------
All looks good, but I'm not sure about removing the broken PrivateDataParser test (the class still exists, but is not exported and has a private parseXml method; it could be exported for the test and the test class marked as friend, for instance?). sandsmark, can you confirm or deny removal of the autotest? - David Faure On Feb. 8, 2014, 7:52 p.m., Michael Palimaka wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115586/ > ----------------------------------------------------------- > > (Updated Feb. 8, 2014, 7:52 p.m.) > > > Review request for KDE Frameworks. > > > Repository: attica > > > Description > ------- > > - Use BUILD_TESTING from ECM instead of custom ATTICA_ENABLE_TESTS > - Remove enable_testing since this is done automatically by KDECMakeSettings > - Mark tests > - Remove old test that doesn't build, which tests API that no longer exists > > > Diffs > ----- > > CMakeLists.txt 445503efba2b49546137ce1548b3ddcddb2a0ebb > autotests/CMakeLists.txt e4a455730fd1faddf547b6c91d4477d707f41551 > autotests/privatedatatest.cpp e5c2d2675b1f33631f2b31a2ce0eb5db28f3406a > tests/projecttest/CMakeLists.txt 0fce1bf8b2f9f93b9ee5b7961a12aeecdc8828a8 > > Diff: https://git.reviewboard.kde.org/r/115586/diff/ > > > Testing > ------- > > Builds. Tests pass when enabled, and do not run when not. > > > Thanks, > > Michael Palimaka > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel