<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/108418/">http://git.reviewboard.kde.org/r/108418/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">A similar patch has been submitted today as #108407. I'll have a look at this as well, but ideally you should probably talk to Lukas about it.</pre>
<br />
<p>- Dario</p>
<br />
<p>On January 15th, 2013, midnight UTC, Xuetian Weng wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for kde-workspace, Solid, Dario Freddi, and Lukáš Tinkl.</div>
<div>By Xuetian Weng.</div>
<p style="color: grey;"><i>Updated Jan. 15, 2013, midnight</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">https://bugzilla.redhat.com/show_bug.cgi?id=859227
patch is straight forward, use systemd to suspend and hibernate.
Add new xml since this patch is shared, and looks like it will always built with systemd so no more check is required.
UPower is still being notified from systemd so Resuming signal can still be used.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I don't have swap so only suspend is tested. Problem seems fixed.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>powerdevil/daemon/BackendConfig.cmake <span style="color: grey">(5dbe6f6)</span></li>
<li>powerdevil/daemon/CMakeLists.txt <span style="color: grey">(04f3050)</span></li>
<li>powerdevil/daemon/backends/upower/powerdevilupowerbackend.h <span style="color: grey">(ba942bd)</span></li>
<li>powerdevil/daemon/backends/upower/powerdevilupowerbackend.cpp <span style="color: grey">(97a409b)</span></li>
<li>powerdevil/daemon/backends/upower/upowersuspendjob.h <span style="color: grey">(bbe2f45)</span></li>
<li>powerdevil/daemon/backends/upower/upowersuspendjob.cpp <span style="color: grey">(fa64ab0)</span></li>
<li>powerdevil/daemon/org.freedesktop.login1.Manager.xml <span style="color: grey">(PRE-CREATION)</span></li>
<li>powerdevil/daemon/powerdevilpolicyagent.cpp <span style="color: grey">(1451c5e)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/108418/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>