On Mon, Jul 27, 2009 at 11:15:23PM -0300, Italo Valcy wrote: > > The package can be found on mentors.debian.net: > - URL: > http://mentors.debian.net/debian/pool/main/l/libhtml-calendarmonthsimple-perl > - Source repository: deb-src http://mentors.debian.net/debian unstable > main contrib non-free > - dget > http://mentors.debian.net/debian/pool/main/l/libhtml-calendarmonthsimple-perl/libhtml-calendarmonthsimple-perl_1.25-1.dsc > looks good for the most part. a few minor recommendations though. instead of linking to a screenshot in the long description, you should upload the screenshot to screenshots.debian.net (which you may have to do after it is uploaded). and instead of using an author based URL in debian/watch, which won't work if somebody else uploads a new version of this distribution, you should use something based on http://search.cpan.org/dist/HTML-CalendarMonthSimple/. same goes for the homepage. the homepage should probably be the URL I previously mentioned. -- _________________________ Ryan Niebur ryanryan52@gmail.com
Attachment:
signature.asc
Description: Digital signature