----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124388/ -----------------------------------------------------------
(Updated July 20, 2015, 2:04 p.m.) Review request for Plasma and Solid. Changes ------- Rebased on master, removed consolekit2InhibitFd. When trying to consolidate the code I noticed that daemon/actions/bundled/suspendsession.cpp calls setupSystemdInhibition and we can't easily pass in the QDBusInterface there so I promoted the managerIface to a member variable that could then be used. Repository: powerdevil Description ------- Internally, it uses the login1 interface with ConsoleKit2 since they share identical DBUS calls. It also changes the setupSystemdInhibition to setupSeatManagerInhibition so that the login1 and ConsoleKit2 inhibit calls to minimize code duplication. This patch doesn't modify the build system to keep it small. A future patch can be added to use ConsoleKit2 on BSD systems as well. ConsoleKit2's suspend/hibernate/inhibit DBUS API is documented at: http://consolekit2.github.io/ConsoleKit2/#Manager Diffs (updated) ----- daemon/backends/upower/powerdevilupowerbackend.h 3752977 daemon/backends/upower/powerdevilupowerbackend.cpp 5120d3e daemon/powerdevilpolicyagent.h ef41ad7 daemon/powerdevilpolicyagent.cpp 9210a06 Diff: https://git.reviewboard.kde.org/r/124388/diff/ Testing ------- Built powerdevil with patch and pkla files on a Gentoo system with OpenRC. Before patch there was no suspend option, after there was and clicking it resulted in the system properly suspending. Thanks, Eric Koegel
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
