Review Request 123042: Port away from KServiceTrader

Aleix Pol Gonzalez aleixpol at kde.org
Thu Mar 19 02:49:36 UTC 2015


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

(Updated March 19, 2015, 3:49 a.m.)


Review request for kdeconnect and Alex Richardson.


Repository: kdeconnect-kde


Description
-------

Uses KPluginLoader+KPluginMetadata instead.
Describes plugins in json instead of desktop files. These desktop files are then embedded into the .so file. All the plugins will be in a kdeconnect/ directory, and those will be the ones to look for.

Note it doesn't drop the KService dependency as KIOCore is a KDE Connect dependency as well.


Diffs
-----

  CMakeLists.txt b81a476 
  KDEConnectMacros.cmake PRE-CREATION 
  core/daemon.h 70f9abb 
  core/device.cpp 0845c45 
  core/pluginloader.h e92b73e 
  core/pluginloader.cpp b6f0139 
  interfaces/CMakeLists.txt 78cc9f2 
  kcm/kcm.cpp 806d99f 
  plugins/battery/CMakeLists.txt d2a4237 
  plugins/battery/batteryplugin.cpp 3202156 
  plugins/battery/kdeconnect_battery.desktop 9601e04 
  plugins/battery/kdeconnect_battery.json PRE-CREATION 
  plugins/clipboard/CMakeLists.txt 3f8ef3c 
  plugins/clipboard/clipboardplugin.cpp f9b2715 
  plugins/clipboard/kdeconnect_clipboard.desktop 94faa78 
  plugins/clipboard/kdeconnect_clipboard.json PRE-CREATION 
  plugins/mousepad/CMakeLists.txt a0994fb 
  plugins/mousepad/kdeconnect_mousepad.desktop d6c68cf 
  plugins/mousepad/kdeconnect_mousepad.json PRE-CREATION 
  plugins/mousepad/mousepadplugin.cpp 958a288 
  plugins/mpriscontrol/CMakeLists.txt f10c502 
  plugins/mpriscontrol/kdeconnect_mpriscontrol.desktop e9d4e3c 
  plugins/mpriscontrol/kdeconnect_mpriscontrol.json PRE-CREATION 
  plugins/mpriscontrol/mpriscontrolplugin.cpp 7d705e7 
  plugins/notifications/CMakeLists.txt 9f56471 
  plugins/notifications/kdeconnect_notifications.desktop 040f5fe 
  plugins/notifications/kdeconnect_notifications.json PRE-CREATION 
  plugins/notifications/notificationsplugin.cpp 9af82e1 
  plugins/pausemusic/CMakeLists.txt 2f9f42b 
  plugins/pausemusic/kdeconnect_pausemusic.desktop 31e40d3 
  plugins/pausemusic/kdeconnect_pausemusic.json PRE-CREATION 
  plugins/pausemusic/pausemusicplugin.cpp b6cf34a 
  plugins/ping/CMakeLists.txt 7628fd8 
  plugins/ping/kdeconnect_ping.desktop 46959a1 
  plugins/ping/kdeconnect_ping.json PRE-CREATION 
  plugins/ping/pingplugin.cpp 519bee5 
  plugins/screensaver-inhibit/CMakeLists.txt f7bda44 
  plugins/screensaver-inhibit/kdeconnect_screensaver_inhibit.desktop d2773fd 
  plugins/screensaver-inhibit/kdeconnect_screensaver_inhibit.json PRE-CREATION 
  plugins/screensaver-inhibit/screensaverinhibitplugin.cpp 4d74fb4 
  plugins/sftp/CMakeLists.txt 1f5123c 
  plugins/sftp/kdeconnect_sftp.desktop 7ad412c 
  plugins/sftp/kdeconnect_sftp.json PRE-CREATION 
  plugins/sftp/sftpplugin.cpp 88f8a36 
  plugins/share/CMakeLists.txt 7016e26 
  plugins/share/kdeconnect_share.desktop 7433061 
  plugins/share/kdeconnect_share.json PRE-CREATION 
  plugins/share/shareplugin.cpp 049f57d 
  plugins/telephony/CMakeLists.txt 8db5629 
  plugins/telephony/kdeconnect_telephony.desktop d44b595 
  plugins/telephony/kdeconnect_telephony.json PRE-CREATION 
  plugins/telephony/telephonyplugin.cpp e8d9ad5 

Diff: https://git.reviewboard.kde.org/r/123042/diff/


Testing
-------

All plugins are still loaded.


Thanks,

Aleix Pol Gonzalez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20150319/5f3d3882/attachment-0001.html>


More information about the KDEConnect mailing list