<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/124120/">https://git.reviewboard.kde.org/r/124120/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Just one change: Remove the CMake macro "generate_and_install_dbus_interface" if we don't use it anymore, and move "geninterface" to that file, so that we can use it in more places (and easily copied over to other projects). You will need to add an extra parameter to "geninterface", to specify the sources variable to add the interface to.</pre>
<br />
<p>- Albert Vaca Cintora</p>
<br />
<p>On June 17th, 2015, 8:49 p.m. PDT, Aleix Pol Gonzalez wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for kdeconnect.</div>
<div>By Aleix Pol Gonzalez.</div>
<p style="color: grey;"><i>Updated June 17, 2015, 8:49 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kdeconnect-kde
</div>
<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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Simplifies the code altogether by centralizing the generation into 1 call that does all the steps.</p></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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Just works.</p></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>cmake/DbusInterfaceMacros.cmake <span style="color: grey">(2c49753)</span></li>
<li>core/CMakeLists.txt <span style="color: grey">(4d75251)</span></li>
<li>interfaces/CMakeLists.txt <span style="color: grey">(bae0cff)</span></li>
<li>plugins/battery/CMakeLists.txt <span style="color: grey">(cfb799a)</span></li>
<li>plugins/mprisremote/CMakeLists.txt <span style="color: grey">(0392249)</span></li>
<li>plugins/notifications/CMakeLists.txt <span style="color: grey">(233fae1)</span></li>
<li>plugins/sftp/CMakeLists.txt <span style="color: grey">(7f68e0c)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/124120/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>