Hi Warren, Thanks for making all the suggested changes in -1~pre3. Other than the following 3 points, the packages look good. 1) I have to tell you that I made an error in the clean target I suggested for debian/rules: > for file in netcdf-c netcdf-cxx netcdf-f77 netcdf-f90 \ > netcdf-install netcdf-tutorial netcdf ; do \ > rm -f man/$file.ps man/$file.pdf \ > man/$file.html man/$file.dvi ; \ > done > rm -f ncdump/ctest.c ncdump/ctest64.c ... specifically, all of the $ signs above should be changed to $$ in order to escape them from Make. My apologies for that! 2) There is another new thing that appeared in the -1~pre3 release, which is that now the following appears in the diff.gz: > $ lsdiff -z netcdf_3.6.2-1~pre3.diff.gz | grep -v /debian/ > netcdf-3.6.2/man/netcdf/index.html > netcdf-3.6.2/man/netcdf/Foreword.html > netcdf-3.6.2/man/netcdf/Summary.html [snipped 341 more lines in "man" directory] Where did those come from? Is their addition in the diff intentional? I'm guessing that these may be HTML versions of the documentation that were generated at some point, but not cleaned when you ran dpkg-buildpackage. In this case maybe the above clean target should be extended by adding this: rm -rf man/$$file ; \ within the for loop. Also, do you plan to install these HTML files in the documentation directory as well? 3) And of course please let me know what decision you reach about these points: * possibility to build an arch:all documentation package * whether or not this is the same as the "netcdf-doc" referenced by #321337 If not, and if you do decide to create the arch:all package, I guess it should be named something other than "netcdf-doc" since various packages (sbuild, apt) don't deal well with binary packages of the same name as an unrelated source package [0]. [0] http://lists.debian.org/debian-policy/2007/01/msg00031.html best regards, -- Kevin B. McCarty <kmccarty@princeton.edu> Physics Department WWW: http://www.princeton.edu/~kmccarty/ Princeton University GPG: public key ID 4F83C751 Princeton, NJ 08544
Attachment:
signature.asc
Description: OpenPGP digital signature