<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;">Man, how you do this :) Check my review request posted earlier today at https://git.reviewboard.kde.org/r/108407/</pre>
 <br />









<p>- Lukáš</p>


<br />
<p>On January 15th, 2013, 1 a.m. CET, 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, 1 a.m.</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>