Review Request 120373: Port all CMake and source files to the new Grantlee5 library
Kevin Funk
kfunk at kde.org
Thu Sep 25 22:15:31 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120373/#review67460
-----------------------------------------------------------
Ship it!
CMakeLists.txt
<https://git.reviewboard.kde.org/r/120373/#comment47060>
The 'Grantlee' inside set_target_properties needs to be adapted as well.
Thanks for fixing!
- Kevin Funk
On Sept. 25, 2014, 9:42 p.m., Simon Wächter wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120373/
> -----------------------------------------------------------
>
> (Updated Sept. 25, 2014, 9:42 p.m.)
>
>
> Review request for KDevelop and Kevin Funk.
>
>
> Repository: kdevplatform
>
>
> Description
> -------
>
> A previous commit broke the build process if Grantlee5 is available. This patch updates all CMake files to the new syntax and variables. Also the Grantlee Ptr class was removed and should be replaced with a QSharedPointer.
>
>
> Diffs
> -----
>
> CMakeLists.txt 903e4a0
> language/CMakeLists.txt 6a3a29f
> language/codegen/templateengine.cpp 991eb1e
> plugins/CMakeLists.txt 2497765
> template/CMakeLists.txt a35d2cd
>
> Diff: https://git.reviewboard.kde.org/r/120373/diff/
>
>
> Testing
> -------
>
> Testing was done under a project neon 5 system:
>
> - With installed cmake, includes and libraries
> - Without cmake, includes and libraries
>
> Both situations works. Further testing (just to be sure) is welcomed.
>
>
> Thanks,
>
> Simon Wächter
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20140925/2cbb6041/attachment.html>
More information about the KDevelop-devel
mailing list