Re: voro++ package
Roger Wesson <rw@nebulousresearch.org> writes:
> Great! Then it's all ready to be reviewed.
OK, here are my comments:
* Change order in d/copyright: first the common entry (*), then the more
specialized.
* Add VCS entries in d/control
* html/jquery.js has no source. Don't just overwrite the lintian tag for this.
Since the html subdir is not used anyway, I would recommend removing it
from the tarball completely:
- "Files-Excluded:" entry in the d/copyright header.
- add "opts=dversionmangle=s/\+dfsg\d*//,repacksuffix=+dfsg" to debian/watch
- if a doc is useful, create it yourself with doxygen
* provide some description to the patches in debian/patches
* install the NEWS file as changelog (with dh_installchangelogs)
* install the examples (with dh_installexamples)
* please clarify with upstream whether a shared library would be
useful here, and if yes, povide one. In that case the package should
be split into one package for the lib, one for the -dev tools and
one for the utility. See Debian Policy 8.3
* Lintian warns about
manpage-has-errors-from-man usr/share/man/man1/voro++.1.gz 85:
warning: numeric expression expected (got `s')
Since you are already patching the manpage, maybe you can fix that?
* I added voronoi to the debian-astro blend, however it may be worth to
add it to the "mathematics" task of debian science as well. If you
want that, you should clone the debian-science blends package
ssh://git.debian.org/git/blends/projects/science.git
and add your package in tasks/mathematics (resp. mathematics-dev for
the debelopment package).
Best regards
Ole
Reply to: