On Monday 04 August 2008 09:38:01 Ding Honghui wrote:
> http://mentors.debian.net/debian/pool/main/c/curlftpfs/curlftpfs_0.9.2-1.dsc
I've some comments/questions:
* The packages uses an obsolete Standards-Version. Please adapt it to the
current Debian Policy (3.8.0). You can use this list to review what needs
to be changed: /usr/share/doc/debian-policy/upgrading-checklist.txt.gz
* The Build-Depends lines in debian/control and debian/control.in differ, see
libcurl*-dev. This only adds confusion. Those build-dependencies should be
the same or if you don't need the control.in file you can get rid of it.
* There are still the Vcs-{Svn,Browser} fields that point to
svn://svn.debian.org/collab-maint/deb-maint/curlftpfs/trunk/ and
http://svn.debian.org/wsvn/collab-maint/deb-maint/curlftpfs/trunk/, but you
obviously don't use this repository. If you don't use this, remove them.
I'd however prefer if you could use this repository so that the history of
this package does not get lost.
* curlftpfs now requires curl (>= 7.17.0), see configure.ac. Have you checked
if the build-dependency in debian/control needs to be updated?
Cheers,
Frank
Attachment:
signature.asc
Description: This is a digitally signed message part.