Review Request 124388: Add ConsoleKit2 as a power backend

Eric Koegel eric.koegel at gmail.com
Wed Jul 22 17:08:43 UTC 2015


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

(Updated July 22, 2015, 5:08 p.m.)


Review request for Plasma and Solid.


Changes
-------

I used the QScopedPointer, really nice feature! I also corrected the if (!m_login1Interface.. Thanks!


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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150722/2889fa94/attachment-0001.html>


More information about the Plasma-devel mailing list