On Sat, Aug 8, 2009 at 4:09 PM, Joachim Reichel<joachim.reic...@gmx.de> wrote: > Hi Mathieu, > >>> I am looking for a sponsor for my package "vera++". >> >> I might be interested in sponsoring this package. I'll have a closer >> look at the weekend. > > I found a couple of issues in your package: > > - Your package does not build in an up-to-date pbuilder sid environment. > Please test your package with pbuilder. > > - There are a couple of lintian messaged (info level). Although not > warnings or errors it would be nice to fix them: > > I: vera++ source: quilt-patch-missing-description vera.patch > I: vera++: hyphen-used-as-minus-sign usr/share/man/man1/vera++.1.gz:51 > (and many more) > I: vera++: package-contains-empty-directory usr/sbin/ > I: vera++: copyright-with-old-dh-make-debian-copyright > > - The package contains a copy of cpptcl 1.1.4 from the same author (see > the list archives for discussions about embedded copies of other > libraries). Since cpptcl is not yet packaged for Debian I would accept > this for now (in particular since this version 1.1.4 does not seem to be > available from the project page yet). > > - debian/copyright: You should mention the GPL version for the Debian > packaging (and adjust the link if necesary). > > - debian/watch: Does not work (use uscan to test). > > - debian/dirs: Remove /usr/sbin. > > - debian/README.Debian: Not really necessary since it just describes > standard modifications to the build process (in contrast to reasons for > repackaging the source for example). On the other hand, it does not hurt. > > - debian/changelog, README.Debian: Please update the date (dch -a). > > - debian/rules: Please remove the dummy comments and commented lines. > > - manpage: The first five occurrences of VERA++ refer directly to the > executable and should be in all-lowercase. Please fix the layout of the > OPTIONS section (have a look at other manpages). Please change the > synopsis section to describe the basic invocation syntax (see other > manpages). The man page should mention the location of the example scripts. > > - /usr/share/doc/vera++: Do not install README.Debian (useless for the > binary package) and LICENSE_1_0.txt (contained in copyright).
I made a new upload which I believe fixes all of the above. Thanks again. $ dupload -t mentors ../vera++_1.1.1-2_amd64.changes dupload note: no announcement will be sent. Checking signatures before upload......signatures are ok Uploading (ftp) to mentors.debian.net:/ [ job vera++_1.1.1-2_amd64 from vera++_1.1.1-2_amd64.changes vera++_1.1.1-2_amd64.deb, size ok, md5sum ok, sha1sum ok, sha256sum ok vera++_1.1.1-2.dsc, size ok, md5sum ok, sha1sum ok, sha256sum ok vera++_1.1.1-2.diff.gz, size ok, md5sum ok, sha1sum ok, sha256sum ok vera++_1.1.1-2_amd64.changes ok ] Uploading (ftp) to mentors (mentors.debian.net) + FTP passive mode selected [ Uploading job vera++_1.1.1-2_amd64 vera++_1.1.1-2_amd64.deb 111.4 kB, ok (2 s, 55.70 kB/s) vera++_1.1.1-2.dsc 1.2 kB, ok (0 s, 1.16 kB/s) vera++_1.1.1-2.diff.gz 6.3 kB, ok (2 s, 3.14 kB/s) vera++_1.1.1-2_amd64.changes 1.5 kB, ok (1 s, 1.47 kB/s) ] -- Mathieu -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
