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

(Updated Dub. 27, 2015, 9:44 dop.)


Status
------

This change has been discarded.


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

Reply via email to