Bug#840739: RFS: gudhi/1.3.1+ds-1 [ITP]
On Thu, Feb 9, 2017 at 4:47 AM, Gard Spreemann wrote: > On Wednesday, 8 February 2017 18:46:46 CET Roger Shimizu wrote: >> Dear Gard, >> >> I cannot sponsor the upload. But here's my review that I hope it's helpful. > > Dear Roger, > > Thank you very much for your helpful feedback. I believe I have > rectified some of the below. > >> Here're the items need to be fixed: >> - missing in debian/copyright, not GPL-3+ license: >> cmake/modules/FindEigen3.cmake >> cmake/modules/FindTBB.cmake >> include/gudhi/Contraction/CGAL_queue/*.h >> data/points/COIL_database/images/* >> doc/*/*.png >> Better to ask upstream to confirm license of those image files. >> Usually license of image files is different from the code. If it's not >> sure simply remove it from "debian source" repack. > > Good catch! I'm sorry for overlooking this. I'll get to work > clarifying the licenses and/or stripping out these. I mean there're src files under other than GPL-3+, which is totally fine. But you need to sort them out, list all the licenses and their src files. Please take a look at my package as example: - https://anonscm.debian.org/git/collab-maint/shadowsocks-libev.git/tree/debian/copyright >> - lintian reports: >> I: libgudhi-dev: spelling-error-in-copyright unneccessary unnecessary > > Fixed. > >> Other comments, nice to have: >> - it's more convenient if you can export your work to some modern >> SCM, such as git >>the review will be easier if doing with such SCM >>you can omit the final releasing commit, so if there's something >> still need to work, you don't have to push forcefully. > > Done; https://git.nonempty.org/debian-gudhi/ > >> - add Vcs-* line to d/control (depends on the above item) > > Done. > >> - bump to debhelper 10 > > Done. > >> - wrap and sort Build-Depends & Depends list in d/control > > Done. Great! On Tue, Feb 14, 2017 at 7:13 PM, Gard Spreemann wrote: > On Wednesday, 8 February 2017 20:47:07 CET Gard Spreemann wrote: >> On Wednesday, 8 February 2017 18:46:46 CET Roger Shimizu wrote: >> > >> > - have separated -doc package >> >> The documentation shipped with upstream's source is rather >> limited. They instead ship a dedicated tarball for documentation >> [1]. I intend to package it too, and have it provide the -doc >> package. >> >> Does this sound sensible to you? > > Scratch that. My package now generates the doxygen documentation and > builds a -doc package. Great! >> I'll upload a new version to mentors.debian.net when I hear back >> from upstream regarding the missing copyrights. > > Upstream say they will take into account my remarks regarding > licensing for the next release. I'll upload a new version to mentors > when that happens. I guess only image files need to confirm, right? We can wait for the version you're satisfied with. > The version on https://git.nonempty.org/debian-gudhi/ has diverged a > bit from the mentors one meanwhile. Most importantly, it now builds a > -doc and an -examples package. Any comments would be greatly > appreciated. No problem about the divergence, git repo is just easy to track what you've changed. Look forward to your updated upload next time. Thanks for your effort! Cheers, -- Roger Shimizu, GMT +9 Tokyo PGP/GPG: 4096R/6C6ACD6417B3ACB1
Bug#840739: RFS: gudhi/1.3.1+ds-1 [ITP]
On Wednesday, 8 February 2017 20:47:07 CET Gard Spreemann wrote: > On Wednesday, 8 February 2017 18:46:46 CET Roger Shimizu wrote: > > > > - have separated -doc package > > The documentation shipped with upstream's source is rather > limited. They instead ship a dedicated tarball for documentation > [1]. I intend to package it too, and have it provide the -doc > package. > > Does this sound sensible to you? Scratch that. My package now generates the doxygen documentation and builds a -doc package. > I'll upload a new version to mentors.debian.net when I hear back > from upstream regarding the missing copyrights. Upstream say they will take into account my remarks regarding licensing for the next release. I'll upload a new version to mentors when that happens. The version on https://git.nonempty.org/debian-gudhi/ has diverged a bit from the mentors one meanwhile. Most importantly, it now builds a -doc and an -examples package. Any comments would be greatly appreciated. Best regards, Gard
Bug#840739: RFS: gudhi/1.3.1+ds-1 [ITP]
On Wednesday, 8 February 2017 18:46:46 CET Roger Shimizu wrote: > Dear Gard, > > I cannot sponsor the upload. But here's my review that I hope it's helpful. Dear Roger, Thank you very much for your helpful feedback. I believe I have rectified some of the below. > Here're the items need to be fixed: > - missing in debian/copyright, not GPL-3+ license: > cmake/modules/FindEigen3.cmake > cmake/modules/FindTBB.cmake > include/gudhi/Contraction/CGAL_queue/*.h > data/points/COIL_database/images/* > doc/*/*.png > Better to ask upstream to confirm license of those image files. > Usually license of image files is different from the code. If it's not > sure simply remove it from "debian source" repack. Good catch! I'm sorry for overlooking this. I'll get to work clarifying the licenses and/or stripping out these. > - lintian reports: > I: libgudhi-dev: spelling-error-in-copyright unneccessary unnecessary Fixed. > Other comments, nice to have: > - it's more convenient if you can export your work to some modern > SCM, such as git >the review will be easier if doing with such SCM >you can omit the final releasing commit, so if there's something > still need to work, you don't have to push forcefully. Done; https://git.nonempty.org/debian-gudhi/ > - add Vcs-* line to d/control (depends on the above item) Done. > - bump to debhelper 10 Done. > - wrap and sort Build-Depends & Depends list in d/control Done. > - have separated -doc package The documentation shipped with upstream's source is rather limited. They instead ship a dedicated tarball for documentation [1]. I intend to package it too, and have it provide the -doc package. Does this sound sensible to you? > - In favor of https URL over http in debian/copyright Done (for the copyright format URL; upstream's website is not on HTTPS). > So far it's enough for now. Thanks! I'll upload a new version to mentors.debian.net when I hear back from upstream regarding the missing copyrights. [1] https://gforge.inria.fr/frs/download.php/file/ 36176/2016-09-12-16-12-33_GUDHI_DOC_1.3.1.tar.gz Best, Gard
Bug#840739: RFS: gudhi/1.3.1+ds-1 [ITP]
Control: tag -1 +moreinfo Dear Gard, On Fri, Oct 14, 2016 at 7:32 PM, Gard Spreemann wrote: > > I am looking for a sponsor for my package "gudhi" > > * Package name : gudhi >Version : 1.3.1+ds-1 >Upstream Author : Gudhi Project / INRIA > * URL : http://gudhi.gforge.inria.fr/ > * License : GPL3+ >Section : math > > GUDHI is a C++ header-only template library for computations in the > mathematical field of topological data analysis. I cannot sponsor the upload. But here's my review that I hope it's helpful. Here're the items need to be fixed: - missing in debian/copyright, not GPL-3+ license: cmake/modules/FindEigen3.cmake cmake/modules/FindTBB.cmake include/gudhi/Contraction/CGAL_queue/*.h data/points/COIL_database/images/* doc/*/*.png Better to ask upstream to confirm license of those image files. Usually license of image files is different from the code. If it's not sure simply remove it from "debian source" repack. - lintian reports: I: libgudhi-dev: spelling-error-in-copyright unneccessary unnecessary Other comments, nice to have: - it's more convenient if you can export your work to some modern SCM, such as git the review will be easier if doing with such SCM you can omit the final releasing commit, so if there's something still need to work, you don't have to push forcefully. - add Vcs-* line to d/control (depends on the above item) - bump to debhelper 10 - wrap and sort Build-Depends & Depends list in d/control - have separated -doc package - even upstream didn't provide script to install example, you can install those example files by adding d/docs file - In favor of https URL over http in debian/copyright So far it's enough for now. Cheers, -- Roger Shimizu, GMT +9 Tokyo PGP/GPG: 4096R/6C6ACD6417B3ACB1
Bug#840739: RFS: gudhi/1.3.1+ds-1 [ITP]
Package: sponsorship-requests Severity: wishlist Dear mentors, I am looking for a sponsor for my package "gudhi" * Package name : gudhi Version : 1.3.1+ds-1 Upstream Author : Gudhi Project / INRIA * URL : http://gudhi.gforge.inria.fr/ * License : GPL3+ Section : math GUDHI is a C++ header-only template library for computations in the mathematical field of topological data analysis. My package currently only builds libgudhi-dev, which contains the upstream library headers for this header-only template library. The build process does also build a bunch of example binaries, but these are *not* installed. I intend to add them to a gudhi-examples package at a later time, but upstream does not currently ship good install rules for them. My package includes the following patches: - Stash-flags.patch: Work around an upstream-acknowledged bug where CMake ignores compiler flags. - Don-t-use-static-boost.patch: Don't link examples statically to boost. This is not currently relevant, as the examples are not used. - Disable-some-tests-that-uncleanly-write-outside-the-.patch: Disable two upstream self-tests that write outside the build directory. (I now see that this patch's name was cut off by gbp). The package can be accessed at https://mentors.debian.net/package/gudhi and through dget -x https://mentors.debian.net/debian/pool/main/g/gudhi/gudhi_1.3.1+ds-1.dsc The package builds cleanly against a sid pbuilder. Best, Gard Spreemann -- System Information: Debian Release: stretch/sid APT prefers testing APT policy: (990, 'testing'), (500, 'unstable') Architecture: amd64 (x86_64) Foreign Architectures: i386 Kernel: Linux 4.7.0-1-amd64 (SMP w/4 CPU cores) Locale: LANG=C.UTF-8, LC_CTYPE=C.UTF-8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/dash Init: systemd (via /run/systemd/system)
