Bug 318737

Summary:	Powerdevil doesn't suspend with latest refactoring
Product:	[Unmaintained] solid	Reporter:	Hrvoje Senjan <hrvoje.senjan>
Component:	powermanagement-daemon	Assignee:	Dario Freddi <drf>
Status:	RESOLVED FIXED
Severity:	major	CC:	alinm.elena
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	http://commits.kde.org/kde-workspace/ad805b504eb3ddec063a3e9060a6d07180232a47	Version Fixed/Implemented In:
Sentry Crash Report:

Description Hrvoje Senjan 2013-04-22 20:25:32 UTC

Using latest master. As described in summary. The issue is valid for GUI suspend, 
qdbus org.kde.Solid.PowerManagement /org/kde/Solid/PowerManagement/Actions/SuspendSession org.kde.Solid.PowerManagement.Actions.SuspendSession.suspendToRam

works as expected.  Also qdbus --system org.freedesktop.login1 /org/freedesktop/login1 org.freedesktop.login1.Manager.Suspend true  works.
Using systemd (202 version)

Reproducible: Always

Steps to Reproduce:
1. Try to suspend with krunner/kickoff/etc

Actual Results:  
System doesn't suspend

Expected Results:  
Should suspend

Probably caused by 63e9e0.
Powevdevil says it's initialized
kded(730) PowerDevil::Core::onBackendReady: Backend is ready, KDE Power Management system initialized
kded(730) PowerDevil::Core::onDeviceAdded: A new battery was detected
kded(730) PowerDevil::PolicyAgent::onSessionHandlerRegistered: Session path: "/org/freedesktop/login1/session/_32"
kded(730) PowerDevil::PolicyAgent::onSessionHandlerRegistered: ACTIVE SESSION PATH: "/org/freedesktop/login1/session/_32"
kded(730) PowerDevil::PolicyAgent::onActiveSessionChanged: Current session is now active
kded(730) PowerDevil::PolicyAgent::setupSystemdInhibition: fd passing available: true
kded(730) PowerDevil::PolicyAgent::setupSystemdInhibition: systemd powersave events handling inhibited, descriptor: 22
kded(730) PowerDevil::PolicyAgent::onSessionHandlerRegistered: systemd support initialized

No kdebugs when trying to suspend via kickoff, etc

Comment 1 Hrvoje Senjan 2013-04-24 18:05:48 UTC

Git commit ad805b504eb3ddec063a3e9060a6d07180232a47 by Hrvoje Senjan.
Committed on 24/04/2013 at 20:02.
Pushed by hrvojes into branch 'master'.

Move away suspend, hibernate and ResumingfromSuspend actions from using Solids DBus service to Solid::PowerManagement

Kdelibs provides a method already for them, and Solid's interface
can change over time

REVIEW: 110135

M  +2    -1    kcontrol/randr/module/randrmonitor.cpp
M  +2    -10   ksmserver/screenlocker/greeter/greeterapp.cpp
M  +2    -10   ksmserver/shutdowndlg.cpp
M  +7    -34   plasma/desktop/applets/kickoff/core/itemhandlers.cpp
M  +1    -1    plasma/generic/dataengines/time/CMakeLists.txt
M  +3    -1    plasma/generic/dataengines/time/timeengine.cpp
M  +2    -2    plasma/generic/runners/powerdevil/PowerDevilRunner.cpp

http://commits.kde.org/kde-workspace/ad805b504eb3ddec063a3e9060a6d07180232a47