Review Request 110422: Changes to the cmake_qt4core_qtest template

Aleix Pol Gonzalez aleixpol at kde.org
Tue May 14 20:13:26 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110422/#review32529
-----------------------------------------------------------

Ship it!


Ok great, I'll push it later if nobody does first.

- Aleix Pol Gonzalez


On May 14, 2013, 6:41 p.m., Adrián Chaves Fernández wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110422/
> -----------------------------------------------------------
> 
> (Updated May 14, 2013, 6:41 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Description
> -------
> 
> I’ve performed the following changes on the template:
> * Moved the source files into an ‘src’ folder, instead of having them in the main folder.
> * Added a missing ${QT_QTCORE_LIBRARY} to the test’s target_link_libraries() CMake command which was preventing the template to build here.
> * Specified the QtCore and QtTest components in the find() CMake command.
> 
> Note: I do *not* have commit access.
> 
> 
> Diffs
> -----
> 
>   app_templates/cpp/CMake/cmake_qt4core_qtest/tests/CMakeLists.txt 42ccb9b 
> 
> Diff: http://git.reviewboard.kde.org/r/110422/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Adrián Chaves Fernández
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20130514/5518a8f9/attachment.html>


More information about the KDevelop-devel mailing list