Change in plasma-framework[master]: Fix the platformstatus kded module

Martin Klapetek (Code Review) noreply at kde.org
Tue Dec 2 18:55:19 UTC 2014


Martin Klapetek has uploaded a new change for review.

  https://gerrit.vesnicky.cesnet.cz/r/197

Change subject: Fix the platformstatus kded module
......................................................................

Fix the platformstatus kded module

The module .desktop file has a typo in the X-KDE-DBus-ModuleName which
results in the KCM not being able to see the status of the module.

Plus it renames the .desktop file to not contain the kded_ prefix, now
it matches every other kded .desktop file we ship (and also matches the
library and dbus module names).

Change-Id: I5006672ca041d9820b4388ba4e99b7899f36be18
---
M src/platformstatus/CMakeLists.txt
R src/platformstatus/platformstatus.desktop
2 files changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.vesnicky.cesnet.cz:29418/plasma-framework refs/changes/97/197/1

diff --git a/src/platformstatus/CMakeLists.txt b/src/platformstatus/CMakeLists.txt
index 6ff2375..43baa3c 100644
--- a/src/platformstatus/CMakeLists.txt
+++ b/src/platformstatus/CMakeLists.txt
@@ -26,6 +26,6 @@
 )
 
 install( TARGETS kded_platformstatus        DESTINATION ${PLUGIN_INSTALL_DIR}/kf5/kded )
-install( FILES kded_platformstatus.desktop  DESTINATION ${SERVICES_INSTALL_DIR}/kded )
+install( FILES platformstatus.desktop  DESTINATION ${SERVICES_INSTALL_DIR}/kded )
 install( FILES org.kde.platformstatus.xml   DESTINATION ${DBUS_INTERFACES_INSTALL_DIR} )
 
diff --git a/src/platformstatus/kded_platformstatus.desktop b/src/platformstatus/platformstatus.desktop
similarity index 98%
rename from src/platformstatus/kded_platformstatus.desktop
rename to src/platformstatus/platformstatus.desktop
index ddb9f0b..09e9153 100644
--- a/src/platformstatus/kded_platformstatus.desktop
+++ b/src/platformstatus/platformstatus.desktop
@@ -2,7 +2,7 @@
 Type=Service
 X-KDE-ServiceTypes=KDEDModule
 X-KDE-Library=kf5/kded/platformstatus
-X-KDE-DBus-ModuleName=plaformstatus
+X-KDE-DBus-ModuleName=platformstatus
 X-KDE-Kded-autoload=true
 X-KDE-Kded-load-on-demand=false
 Name=Platform Status

-- 
To view, visit https://gerrit.vesnicky.cesnet.cz/r/197
To unsubscribe, visit https://gerrit.vesnicky.cesnet.cz/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5006672ca041d9820b4388ba4e99b7899f36be18
Gerrit-PatchSet: 1
Gerrit-Project: plasma-framework
Gerrit-Branch: master
Gerrit-Owner: Martin Klapetek <mklapetek at kde.org>
Gerrit-Reviewer: Ivan Čukić <ivan.cukic at kde.org>


More information about the Plasma-devel mailing list