[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: voro++ package



Hi Roger,

Roger Wesson <rw@nebulousresearch.org> writes:
> Aha, that explains it!  First question, yes, second question,
> no... Build-Depends now includes dh-exec, the package builds, so it's
> ready for review now.

The package looks good, in principle ready for upload. Some minor
things:

* The binary is not linked against the shared lib. This is probably
  because the executable is built (just) before the shared lib. Maybe
  you could change the order here to get the shared lib used?

* It would be useful to create a file debian/upstream/metadata with a
  reference, like Chris H. Rycroft, Voro++: A three-dimensional Voronoi
  cell library in C++, Chaos 19, 041111 (2009) (see
  http://math.lbl.gov/voro++/download/).
  The format is described here: https://wiki.debian.org/UpstreamMetadata
  and an example is here:
  https://sources.debian.net/src/gnuastro/0.2.33-1/debian/upstream/metadata/

* You could consider replacing the "exit()" calls in src/common.hh with
  "abort()", maybe discuss that with upstream. abort() has the advantage
  that one may get a coredump or a stack trace in gdb in case there are
  problems. This could help debugging, if one wants to create own
  programs with libvoro++.

Could you have a look there?

Best regards

Ole


Reply to: