On April 23, 2015, 3:35 p.m., Jan Grulich wrote: > > And yes, this is required as long as NetworkManager headers are included by > > NetworkManagerQt headers. > > Jan Grulich wrote: > I just found out that installing FindNetworkManager.cmake into CMake > modules is not enough, because it still cannot find it. I guess this would > have to be done according to other FindFoo.cmake modules from > extra-cmake-modules, right?
That would certainly help, but then it's not very nice putting a cmake file into another project's directory. I would suggest to either: * Contribute FindNetworkManager.cmake to extra-cmake-modules * Install it within the NetworkManagerQt cmake directory, together with NetworkManagerQtConfig.cmake. - Aleix ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123477/#review79383 ----------------------------------------------------------- On April 23, 2015, 3:01 p.m., Jan Grulich wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/123477/ > ----------------------------------------------------------- > > (Updated April 23, 2015, 3:01 p.m.) > > > Review request for KDE Frameworks, David Faure, Daniel Vrátil, and Lamarque > Souza. > > > Repository: networkmanager-qt > > > Description > ------- > > I'm not 100% sure how this should work and I couldn't find another framework > doing the same, but Daniel Vrátil pointed me out that NetworkManagerQt should > mention in KF5NetworkManagerQtConfig.cmake file that it requires > NetworkManager as dependency. Given this, we also need to install > FindNetworkManager.cmake into CMake modules so NetworkManager can be found by > find_dependency() macro, or FindNetworkManager.cmake can go into > extra-cmake-modules. > > If this is how it should be done, then similar patch would be needed for > ModemManagerQt. > > > Diffs > ----- > > CMakeLists.txt c9e3274 > KF5NetworkManagerQtConfig.cmake.in cdabe8e > > Diff: https://git.reviewboard.kde.org/r/123477/diff/ > > > Testing > ------- > > > Thanks, > > Jan Grulich > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
