Review Request 123349: NetworkManagerQt: comment out unneeded lines from example/CMakeLists.txt
David Faure
faure at kde.org
Mon Apr 13 07:13:22 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123349/#review78882
-----------------------------------------------------------
examples/CMakeLists.txt (line 5)
<https://git.reviewboard.kde.org/r/123349/#comment53936>
well i thought maybe someone wants to see how to use this in their app... but ok IIRC it also shows that on api.kde.org
- David Faure
On avr. 12, 2015, 7:57 après-midi, David Faure wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123349/
> -----------------------------------------------------------
>
> (Updated avr. 12, 2015, 7:57 après-midi)
>
>
> Review request for KDE Frameworks and Jan Grulich.
>
>
> 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/20150413/aa012fb2/attachment.html>
More information about the Kde-frameworks-devel
mailing list