<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/104200/">http://git.reviewboard.kde.org/r/104200/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On March 9th, 2012, 2:23 p.m., <b>Kevin Kofler</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I think adding a dependency is a much cleaner solution than bundling. But I wonder whether the best long-term fix wouldn't be to just move kxmlrpcclient to kdelibs, it clearly seems to be useful for more than just PIM. But in distros, I think most of us prefer the dependency on kdepimlibs (we can also make a subpackage for that library in our packaging) to bundling (which is against Fedora and Debian policies, at least).</pre>
</blockquote>
<p>On March 9th, 2012, 6:21 p.m., <b>Michael Pyne</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I would also agree with moving the functionality of kxmlrpcclient to kdelibs if it is to be used by something as core as the bug reporter (and is useful generically beyond that nowadays, many more things use XML-RPC web services than just Kolab. Plasma applets could use this as well for instance).
I'm not sure it's a good idea to do this move in the 4.7 branch of kdelibs though... is there a way to have kdelibs-4.7 and kdepimlibs-4.8 *both* include the class (with kdepimlibs-4.8 using a weak symbol or something similar to defer to the kdelibs-4.7 one if present)? If that's at all difficult I'd say just add the short-term dependency on kdepimlibs and be done with it.</pre>
</blockquote>
<p>On March 9th, 2012, 6:41 p.m., <b>David Faure</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">You mean kdelibs-4.8, not 4.7.
But 4.8.0 and 4.8.1 have been released already, so IMHO this is something for 4.9.
Meanwhile we can use the duplication solution from this patch.
</pre>
</blockquote>
<p>On March 10th, 2012, 12:16 a.m., <b>George Kiagiadakis</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Yes, as I said I did this bundling only to avoid doing big changes in the 4.8 branch. In 4.9 I agree that we should handle this differently, either move kxmlrpcclient to kdelibs or make kde-runtime depend on kdepimlibs.</pre>
</blockquote>
<p>On March 13th, 2012, 12:31 p.m., <b>Christoph Feck</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">There were compatibility fixes in bugs.kde.org, so do we still need this? I guess using xmlrpc is the more correct way to do it, so we don't break it again later, but maybe this can be postponed to 4.9+ only.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Yes, we don't need it anymore in the 4.8 branch. This is 4.9+ now. What remains to be seen is what to do with kxmlrpcclient...</pre>
<br />
<p>- George</p>
<br />
<p>On March 9th, 2012, 12:45 a.m., George Kiagiadakis wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for KDE Runtime.</div>
<div>By George Kiagiadakis.</div>
<p style="color: grey;"><i>Updated March 9, 2012, 12:45 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;">This patch partially ports DrKonqi to use xml-rpc, which is done at this point to support the new bugzilla 4.2 instance on bugs.kde.org. I have tried to keep the changes as minimal as possible, so that this can be applied to the 4.8 branch.
Note that this patch involves copying the kxmlrpcclient library from kdepimlibs in drkonqi's source code tree. I have done this to avoid modifying dependencies in 4.8.2. However, if an extra dependency of kde-runtime on kdepimlibs sounds ok, I can remove the internal copy in master.
Full history at:
http://quickgit.kde.org/index.php?p=clones%2Fkde-runtime%2Fgkiagia%2Fdrkonqi.git&a=shortlog&h=refs/heads/drkonqi-xmlrpc</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;">Sucessfully reported and attached extra backtrace on:
https://bugs4.kde.org/show_bug.cgi?id=294820
https://bugs4.kde.org/show_bug.cgi?id=294821
The duplicate search also seems to be working fine.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=295276">295276</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>drkonqi/CMakeLists.txt <span style="color: grey">(590239dfbd9236463ef11eede699eb84f5806b7a)</span></li>
<li>drkonqi/bugzillalib.h <span style="color: grey">(c89c85c8df3724dc6e51ed03aab7ef7362a22472)</span></li>
<li>drkonqi/bugzillalib.cpp <span style="color: grey">(2df29bd975196b13df02934bfa8899e30d9627f6)</span></li>
<li>drkonqi/drkonqi_globals.h <span style="color: grey">(9281b0574737632782c96f90dadad1c86abebd0c)</span></li>
<li>drkonqi/kxmlrpc/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>drkonqi/kxmlrpc/README <span style="color: grey">(PRE-CREATION)</span></li>
<li>drkonqi/kxmlrpc/client.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>drkonqi/kxmlrpc/client.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>drkonqi/kxmlrpc/query.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>drkonqi/kxmlrpc/query.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>drkonqi/reportassistantpages_bugzilla.cpp <span style="color: grey">(fba91b6d179c867d9984d2296b7302a469c8d0e5)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/104200/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>