From kde-core-devel  Tue Apr 05 22:28:21 2011
From: Raphael Kubo da Costa <kubito () gmail ! com>
Date: Tue, 05 Apr 2011 22:28:21 +0000
To: kde-core-devel
Subject: Re: Review request: Kcmgrub2
Message-Id: <87hbacz62i.fsf () gmail ! com>
X-MARC-Message: https://marc.info/?l=kde-core-devel&m=130204256200538

Alberto Mattea <alberto@mattea.info> writes:

> Hi all,
> after 4 releases I think kcmgrub2 has reached an acceptable level of maturity,
> so I'd ask for a move to kdereview. It is currently in playground-sysadmin
> (git).

I only took a quick look, as my Py{Qt,KDE}-fu is not that good.

Buildsystem-wise:

  * I did not understand why you used include() instead of
find_package() in, for example,

      include(FindPyQt4)

  * It's probably a good idea to add some kind of README for packagers
explaining what the dependencies are.

Licensing-wise:

  * Don't you need to add the appropriate license header to your code
files?

As for kcmgrub2.py itself:

  * It might be better to set the WhatsThis values in the .ui file
itself.
  * `x' is not a good name for a global variable.