Hi, I have just uploaded new packaging files for voro++ which should
address all the comments below. The package builds, but there are
lintian warnings and errors that I don't know how to get rid of because
I do not know how to properly create the symlink that ldconfig would
create. Whatever I try gives lintian errors or warnings. Right now
there are binaries-have-file-conflict warnings, which I think is because
I'm creating the symlinks at the wrong time, and a
non-dev-pkg-with-shlib-symlink, which I am not sure if it matters or
not, because if there is no symlink I get an error.
So, can someone offer any assistance here and tell me where I'm going
wrong?
Thanks,
Roger
On 10/11/16 20:45, Ole Streicher wrote:
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