Bug#566880: Debian #566880: AllTray in Debian (and Ubuntu)
On Wed, Mar 10, 2010 at 14:23, Raphael Hertzog <hertzog@debian.org> wrote:
> Hi,
Hello Raphael,
> No need to upload the package to mentors.debian.net, I'll rebuild from SVN
> anyway.
Duly noted.
> For the next update, you could use "dh --with autotools_dev", it's a recent
> addition to autotools-dev so you will have to adjust the build dependency.
> That way you can get rid of debian/clean and the override in
> debian/rules.
>
> Other small remarks:
> - the name of the patch is "01_minor_type.diff" instead of
> "01_minor_typo.diff"
Done.
> - the Description field in that patch is not correct, you must put
> something in the first line (short description)
Done.
> - you have not indicated that the patch has been forwarded, did you
> forward it?
The has been forwarded and integrated upstream. I have added an
indication in the patch header.
> - the category "Applications/Graphics" in the menu is certainly not
> the good category, alltray is not about manipulating images...
I agree with you. I have checked the Menu/Section documentation [1],
but I can't seem to find a category where alltray would fit. I would
very much appreciate a suggestion. :)
> I have uploaded your package despite those small remarks, it's good work
> overall. Just fix those details for the next update.
Thank you very much for this.
Cheers,
Ignace M
[1] http://www.debian.org/doc/packaging-manuals/menu.html/ch3.html#s3.5
Reply to: