Hi Christian, Christian Pohl wrote: > I've gone through the messages and uploaded the new package to > mentors.debian.net. > Could you check it?
Thanks for the corrections you made. I still have a few comments though: - lintian isn't completely happy yet: I: webhoneypot: using-first-person-in-description line 5: we I: webhoneypot: using-first-person-in-description line 5: We I: webhoneypot: using-first-person-in-description line 7: We I: webhoneypot: using-first-person-in-description line 7: we I: webhoneypot: using-first-person-in-description line 8: we I: webhoneypot: using-first-person-in-description line 9: we P: webhoneypot: no-upstream-changelog W: webhoneypot: script-not-executable ./usr/share/webhoneypot/update/update-templates.php W: webhoneypot: maintainer-script-empty preinst - You don't seem to have taken Ansgar's remark into account ("I would not expect packages to install a virtual host configuration in /etc/apache2"...) - There's an empty debian/webhoneypot.linitian-overrides file; delete it. - Your debian/watch file is empty too; you should fix it or remove it. - man/webhoneypot.conf.1 should be in section 5. - Several of your files have trailing whitespace or extra newlines at the end (at least debian/{README.debian,control,prerm,rules,postrm,patches/debian-changes-0.1.r123-1} do); it would be nice to clean that up. Cheers, -- Benoît Knecht -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/20101118173231.ga31...@debian.lan