Review Request 119992: PackageStructure plugin loading and tidy up of DataEngien plugin loading

Marco Martin notmart at gmail.com
Fri Aug 29 14:56:41 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119992/#review65507
-----------------------------------------------------------

Ship it!


this having both defines should make old users build without problems.
+1 for me..

- Marco Martin


On Aug. 29, 2014, 2:20 p.m., Aaron J. Seigo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119992/
> -----------------------------------------------------------
> 
> (Updated Aug. 29, 2014, 2:20 p.m.)
> 
> 
> Review request for KDE Frameworks and Plasma.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> This patch removes dead code (ContainmentPackage), clean up DataEngine plugin loading and updates PackageStructure plugin loading.
> 
> Open question: which plugin loading mechanism is to be used:
> 
> a) KServiceTypeTrader
> b) KPluginTrader
> c) KPlugintTrader with a KServiceTypeTrade fallback
> d) KServiceTypeTrader with a KPluginTrader fallback
> 
> DataEngine is currently (b), complete with a #define in the header that won't work as expected.
> In this patch PackageStructure is moved to (b) as well. (There is an open RR for a correponding patch set in plasma-workspace).
> Everything else seems to use (a).
> 
> Some plugins are not used in Plasma 5 due to changes with QML, and I guess that some of this confusion is due to the decreased use of some of these plugins. A decision needs to be made, however about what they should be now and in the future.
> 
> 
> Diffs
> -----
> 
>   src/plasma/pluginloader.cpp d2ba5ca2d3a96fe6f1ce26be41df3b0954b924df 
>   src/plasma/private/packages.cpp 5eb6f0021392257634dfd958c940b2945989e48b 
>   src/plasma/private/packages_p.h 0833a4ed1b5704efffccade5e52589878e8b4957 
>   src/plasma/package.cpp 6ad332167bb83c2f794f9f5d059e9f369ad33841 
>   src/plasma/packagestructure.h fb32c22a6e7df1528b3d7a5b30b94c60a85a93e3 
> 
> Diff: https://git.reviewboard.kde.org/r/119992/diff/
> 
> 
> Testing
> -------
> 
> Ran a Plasma Desktop 5 session with these changes.
> 
> 
> Thanks,
> 
> Aaron J. Seigo
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140829/8c396d9a/attachment.html>


More information about the Plasma-devel mailing list