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 <[EMAIL PROTECTED]> Physics Department
WWW: http://www.princeton.edu/~kmccarty/ Princeton University
GPG: public key ID 4F83C751 Princeton, NJ 08544
signature.asc
Description: OpenPGP digital signature
