Moving KScreen and libkscreen to extragear

Daniel Vrátil dvratil at
Fri Oct 18 09:48:53 BST 2013

On Monday 08 of July 2013 19:19:08 David Edmundson wrote:
> Code wise, things looks pretty good.
> Minor comments:
>  - the library is GPL, not LGPL which is the norm for libraries.
> Is this deliberate?

No, I don't think so. I'm just fine with changing it to LGPL (and I guess Alex 
will be too).

>  - Inside kscreen you have a copy of the metadata.desktop file twice.
> Just have the one and install it to the two places.


> You might also want to use the version number from your main CMakeLists.txt
> in your .desktop file too, otherwise updating can get messy.

Fixed, thanks.

> -Generator::biggestOutput
> rename the local variable "total"'s not a total of anything, I had to
> re-read it 10 times to realise the code was correct.

Right, confused me too :-) Fixed.

> In general though +1 from me.

Thanks :)


Daniel Vrátil
KDE Desktop Team
Associate Software Engineer, Red Hat, Inc.

GPG Key: 0xC59D614F6F4AE348
Fingerprint: 4EC1 86E3 C54E 0B39 5FDD B5FB C59D 614F 6F4A E348
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: This is a digitally signed message part.
URL: <>

More information about the kde-core-devel mailing list