[Pardus-devel] [review] review/applications/network/klibido

Ismail Dönmez ismail at pardus.org.tr
Fri Aug 31 03:40:03 EEST 2007


On Thursday 30 August 2007 17:52:27 Amine Chadly wrote:
> Hi !
> Request for merge of klibido (for contrib I guess ^^)
> Thanks in advance !

- db4 is in system.base, no need to add it as a dep.

- Instead of doing   <Path fileType="data">/usr/share</Path> write             
<Path fileType="data">/usr/share/foobar</Path> (give absolete paths), its 
easier to read and maintain

- Desktop file misses GenericName[] which is used in Pardus menu

- License says 2005-2007, it should be 2007 only

- Path to GPLv2 license text is wrong, its now showing GPLv3

- Leave a newline after autotools.install()

- Instead of pisitools.move(..) use pisitools.insinto

Regards,
ismail

-- 
Bize "yaşayalım" diye verilen hayatı yaşadık... // hayatberbat.com


More information about the Pardus-devel mailing list