Review Request 123477: Add public dependency on NetworkManager

Aleix Pol Gonzalez aleixpol at kde.org
Thu Apr 23 13:35:45 UTC 2015


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



KF5NetworkManagerQtConfig.cmake.in (line 4)
<https://git.reviewboard.kde.org/r/123477/#comment54235>

    nitpick. weird 1 space indentation.


And yes, this is required as long as NetworkManager headers are included by NetworkManagerQt headers.

- Aleix Pol Gonzalez


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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150423/777e3fee/attachment.html>


More information about the Kde-frameworks-devel mailing list