Hi, Alejandro,
I've taken a look at a package you've svn-injected, libfile-chmod-perl
and I have several notes that need fixing before the upload.
Note most of these are not "critical" but I'd still like to see them
addressed before the first upload.
* debian/copyright:
(critical) the license statement is not copied verbatim from the
source. re-wrapping is OK, but the content must be exactly the same.
(cosmetic) the "upstream author" line is usually in the beginning,
right after the "upstream source".
(cosmetic) the "perl is distributed..." note is not part of the
upstream license text and I think it would be better if it is not
indented together with it.
* debian/rules:
you use CDBS. This is a grave bug in my eyes :)
dh-make-perl's templates are an excellent start for writing
debian/rules.
* debian/control:
both short and long description could be improved.
for example:
short: chmod() overrude with symbolic and ls-style modes
long: File::chmod provides an override for the Perl's chmod() function
allowing usage of octal, symbolic or "ls-style" file modes.
(the "see below" part is especially bad as there is no "below")
* the current debhelper compatibility level is 6 (from yesterday :)
* I also changed debian/watch:
+ Split the entry to "directory" and "pattern" lines
+ added v? to the pattern
+ require the version to start with a digit and allow only digits and
periods
I've changed the changelog entry's distribution field to UNRELEASED to
note that the package needs more work. Revert this when you think it is
ready to be reviewed again.
--
dam JabberID: dam@jabber.minus273.org
Attachment:
signature.asc
Description: Digital signature