Review Request 123349: NetworkManagerQt: comment out unneeded lines from example/CMakeLists.txt

David Faure faure at kde.org
Fri Apr 17 12:10:48 UTC 2015


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

(Updated April 17, 2015, 12:10 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and Jan Grulich.


Changes
-------

Submitted with commit 5d5aa966e797976d6159c2695ffc8c86a802397b by David Faure to branch master.


Repository: networkmanager-qt


Description
-------

This is built as part of the framework, so it should not do
find_package on itself, it won't work the first time.

I assume this was there in order to all build separately, so I left
it commented out, but it's wrong to have that in.

Maybe with the last two commits, the examples can be built by default,
like in the other frameworks, so that we ensure that they always do build?


Diffs
-----

  examples/CMakeLists.txt c2484b346b70596ddeb87778c7dd3881851441fb 

Diff: https://git.reviewboard.kde.org/r/123349/diff/


Testing
-------

rebuilt the examples from scratch, worked


Thanks,

David Faure

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150417/6a8a651d/attachment.html>


More information about the Kde-frameworks-devel mailing list