On Fri, 2005-08-26 at 17:13 -0500, Gunnar Wolf wrote:
>
> [snip]
>
> There are still bits of QA to do - For example, you are including a
> README.Debian which reads:
>
> Cvs Perl interface for Debian
> -----------------------------
>
> A full documentation of the module is available in MAN and POD format:
> man Cvs or perldoc Cvs.
>
> Sun, 24 Jul 2005 14:54:59 +0200, Ivo Marino <eim@mentors.debian.net>.
>
> You are also including the module's README itself in the
> /usr/share/doc/libcvs-perl directory - That's not bad, but it is
> useless: Debian ships already built software. Why do you include a
> file that explains how to build this software? (Note to self: Many of
> my modules have this same defect. Fix it :) )
>
I removed the module's README file.
> Well, all that is obvious information. Why do you need it in a
> README.Debian? Leave that file alone for more important matters ;-)
>
What about README.Debian? Should I also remove it completely or just
leave it blank? -- I don't understand what you mean with "leave that
file alone".
> You have a libcvs-perl.substvars file which reads:
>
> perl:Depends=perl (>= 5.6.0-16)
>
> But you are depening on perl >= 5.8.0-7 in debian-control - And
> anyway, the oldest version of Perl in the _stable_ release is
> 5.8.4 (in Sid, 5.8.7)...
>
I'll fix this, too.
> You are not closing an ITP bug with this new upload - Why?
>
Because I have not opened an ITP bug -- Should I do this?
Thanks for further details.
Best regards,
I.
--
.''`. Ivo Marino <eim@mentors.debian.net>
: :' : irc.FreeNode.net chan #debian-mentors
`. `` UIN 32463141 + JID eim@jabber.linux.it
`- Homepage http://mentors.debian.net/~eim/
Attachment:
signature.asc
Description: This is a digitally signed message part