Review Request: Plugin framework version check

Daniele Elmo Domenichelli daniele.domenichelli at gmail.com
Wed Oct 31 15:19:23 UTC 2012


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

Review request for Telepathy, David Edmundson and Lasath Fernando.


Description
-------

[Requires review 107151]*

Add property X-KTp-PluginInfo-Version to plugins
    
This property holds the ktp-text-ui plugin framework version for which the plugin is written, and allow us to switch the branch easily, without caring of removing uncompatible plugins. This also allows us to "disable" plugins if there is a problem just by bumping the version without updating it for that plugin.

I would like to merge this in stable branch, and bump version to 2 in master and to 3 in filters branch


*Not really, I could change it to do it in some other ways, but it is nicer using the version.h file and configuring it in CMakeLists.txt


Diffs
-----

  CMakeLists.txt 1d1d72b877348ade85e4835e8e586b3c18ffc11e 
  filters/emoticons/ktptextui_message_filter_emoticons.desktop ee8a51c23c553e1f1f3383531d79bcef314a3226 
  filters/formatting/ktptextui_message_filter_formatting.desktop 366de49363d72d879c0b3ae5baf442a846a9ab87 
  filters/images/ktptextui_message_filter_images.desktop 6dbbe591535d3202b2be8672fd25d7c63659d183 
  filters/searchexpansion/ktptextui_message_filter_searchexpansion.desktop 8b606c522d97152ad22e1dbcbb3b79db465685de 
  filters/texttospeech/ktptextui_message_filter_tts.desktop 5e9810dc1926e3d2e76c1a0391fc07fab9e43725 
  lib/ktptxtui_message_filter.desktop c7d045337bcfeb7c6bf5096bc8397f756298987b 
  lib/plugin-config-manager.cpp b1a4d5b59a0a63f5b3f2a90ab81ca9a8f4e05693 
  version.h.in PRE-CREATION 

Diff: http://git.reviewboard.kde.org/r/107154/diff/


Testing
-------

Works, only the plugins with the correct version are loaded and displayed in the plugin config dialog


Thanks,

Daniele Elmo Domenichelli

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20121031/455d5700/attachment-0001.html>


More information about the KDE-Telepathy mailing list