[RFC][PATCH] split kutils -> {kprintutils,kcmutils}

David Faure faure at kde.org
Wed Feb 20 15:56:28 GMT 2008


On Wednesday 20 February 2008, Marc Mutz wrote:
> Hi,
> 
> [please CC me on replies, if you want me to see them - thanks]
> 
> Currently, the only reason kutils links against kparts and therefore kio is 
> that it contains kprintpreview.cpp. All the rest is fine with just kdeui.

Yep. I removed the kio+kparts dependencies from kutils before 4.0, but then kprintpreview was added and I didn't notice the re-added dependencies :(

r710818 | dfaure | 2007-09-11 01:08:13 +0200 (Tue, 11 Sep 2007) | 2 lines
KUtils doesn't need KIO anymore

> So, here's a patch to split kutils into
>    1-> kcmutils : kdeui
>    2-> kprintutils : kparts

This is very much like undoing the mistaken addition of kprintpreview indeed,
which should have been in its own lib. Another solution would be to put 
it in kparts but I guess there might be more printing-related classes coming 
up in the future so a separate lib seems to make sense -- libkdeprint anyone? ;)

The "kcmutils" name made me hesitate (kpluginselector.cpp is not related to KCMs), but ok,
it's more or less all about "configuration management" :). And it will stop people from treating
kutils as a dumping ground...

So, no objections from me.

-- 
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).




More information about the kde-core-devel mailing list