Re: RFS: udav

[Paul Wise <pabs@debian.org>]

On Sun, May 3, 2009 at 6:39 AM, Salvatore Bonaccorso
<salvatore.bonaccorso@gmail.com> wrote:

> - dget http://mentors.debian.net/debian/pool/main/u/udav/udav_0.5.1-1.dsc

Some feedback based on the diff.gz:

Please get your package description reviewed by the
debian-l10n-englist email list, it contains some gramatical errors.

Is it really nessecary to repack a tarball just to remove .svn
directories? I suggest just asking upstream to use whatever the
equivalent of 'make dist' is for qmake and using their orig tarball.

It might be a good idea to put an icon in the menu file. From the
upstream screenshots it looks like they have one you could use.

Please add a FreeDesktop menu file too, otherwise udav will not be
visible from the GNOME/LXDE menus.

ChangeLog.txt should be a parameter to dh_installchangelogs instead of
in debian/docs.

The manual page isn't that useful upstream, but they might be
interested in it, with or without some changes.

Lintian complaints:

X: udav: spelling-error-in-binary ./usr/bin/udav usefull useful

-- 
bye,
pabs

http://wiki.debian.org/PaulWise