Review Request: Fix Drkonqi to work with bugzilla 4 (partial port to xml-rpc)

[Michael Pyne]

> On March 9, 2012, 2:23 p.m., Kevin Kofler wrote:
> > 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).

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.


- Michael


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104200/#review11252
-----------------------------------------------------------


On March 9, 2012, 12:45 a.m., George Kiagiadakis wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104200/
> -----------------------------------------------------------
> 
> (Updated March 9, 2012, 12:45 a.m.)
> 
> 
> Review request for KDE Runtime.
> 
> 
> Description
> -------
> 
> 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
> 
> 
> This addresses bug 295276.
>     http://bugs.kde.org/show_bug.cgi?id=295276
> 
> 
> Diffs
> -----
> 
>   drkonqi/CMakeLists.txt 590239dfbd9236463ef11eede699eb84f5806b7a 
>   drkonqi/bugzillalib.h c89c85c8df3724dc6e51ed03aab7ef7362a22472 
>   drkonqi/bugzillalib.cpp 2df29bd975196b13df02934bfa8899e30d9627f6 
>   drkonqi/drkonqi_globals.h 9281b0574737632782c96f90dadad1c86abebd0c 
>   drkonqi/kxmlrpc/CMakeLists.txt PRE-CREATION 
>   drkonqi/kxmlrpc/README PRE-CREATION 
>   drkonqi/kxmlrpc/client.h PRE-CREATION 
>   drkonqi/kxmlrpc/client.cpp PRE-CREATION 
>   drkonqi/kxmlrpc/query.h PRE-CREATION 
>   drkonqi/kxmlrpc/query.cpp PRE-CREATION 
>   drkonqi/reportassistantpages_bugzilla.cpp fba91b6d179c867d9984d2296b7302a469c8d0e5 
> 
> Diff: http://git.reviewboard.kde.org/r/104200/diff/
> 
> 
> Testing
> -------
> 
> 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.
> 
> 
> Thanks,
> 
> George Kiagiadakis
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120309/1f5175cf/attachment.htm>