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