Review Request 119763: Reenable the app wizard and file template wizard
Simon Wächter
waechter.simon at gmail.com
Wed Aug 13 16:21:05 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119763/
-----------------------------------------------------------
(Updated Aug. 13, 2014, 6:21 p.m.)
Review request for KDevelop and Kevin Funk.
Changes
-------
Fixed the problems that Aleix opened. Also added a check if Grantlee for Qt5 does exist - if not, do not enable the two plugins. This is a temporary fix as loong there is no official port of Grantlee.
Repository: kdevplatform
Description
-------
The idea of this patch is to reenable the app wizard and file template wizard of KDevelop in the framework branch. For this, I had to reactivate Grantlee and fix/port away a few other things:
- Reenable Grantlee (Unofficial version 1.0) and change some CMake files (See note)
- Port kdevfilters to the new Qt5 plugin system. The install location was fixed, so the plugin is correctly loaded by Grantlee
- Renable the app wizard and file template wizard plugin. Change the serching and write paths for templates
- Change the default paths of several models. Because the dirs() method of KComponentData is deprecated, I had to introdue methods for adding search paths to the model
- Fixed the unit tests (See testing comment)
Note: Grantlee provides a Qt5 branch called 'volatile_qt5_port. I will ask the current maintainer of Grantlee when this branch will result in a new (official) release
Diffs (updated)
-----
language/codegen/sourcefiletemplate.h 4085476
CMakeLists.txt d73caa8
language/CMakeLists.txt f49664c
language/codegen/sourcefiletemplate.cpp 412b8b9
language/codegen/templateengine.cpp fc7c61e
language/codegen/templatesmodel.h fbd5514
language/codegen/templatesmodel.cpp 5779050
language/codegen/tests/CMakeLists.txt 077add5
language/codegen/tests/codegen_tests_config.h.cmake a963e7b
language/codegen/tests/test_templateclassgenerator.cpp af18ddd
language/codegen/tests/test_templatesmodel.cpp 0a1b65d
plugins/CMakeLists.txt a63447c
plugins/appwizard/CMakeLists.txt fc5a98d
plugins/appwizard/appwizardplugin.cpp 1bb134e
plugins/appwizard/projectselectionpage.h fafad0e
plugins/appwizard/projectselectionpage.cpp ae253f4
plugins/filetemplates/filetemplatesplugin.cpp 8739871
plugins/filetemplates/outputpage.h 347296e
plugins/filetemplates/outputpage.cpp 2b98cb9
template/filters/CMakeLists.txt 5eba65e
template/filters/kdevfilters.h 1a33fc6
template/filters/kdevfilters.cpp b8ffb8a
Diff: https://git.reviewboard.kde.org/r/119763/diff/
Testing
-------
Although this patch was tested under two different project neon systems, I would be very glad if someone can retest it
Thanks,
Simon Wächter
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20140813/a91ad821/attachment.html>
More information about the KDevelop-devel
mailing list