* Mehdi Abaakouk <sil...@sileht.net>, 2012-05-03, 22:26:
http://mentors.debian.net/debian/pool/main/p/python-lzo/python-lzo_1.08-1.dsc
Have you forwarded the patch upstream?
Wouldn't it make sense to omit the "download/" part from the Homepage
field?
Since the licenses for debian/ directory and for the rest of the package
are identical, you could save some space by moving their texts to a
standalone license paragraph.
Could you get rid of the comment in debian/rules?
You don't need --buildsystem=python_distutils; dh with detect the build
system automatically.
Upstream provides a test suite. Please run it at build time, using all
supported Python versions.
pyflakes reports (among others):
./setup.py:24: undefined name 'CURL_DIR'
./setup.py:25: undefined name 'CURL_DIR'
This does not affect Debian, but you might want to report it upstream.
This packages closes ITP #671375
(http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=671375)
As per Developer's Reference §5.1, you should have sent a copy of the
ITP to debian-devel. Also, Version pseudo-header doesn't make sense for
wnpp bugs.
--
Jakub Wilk
--
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/20120504162602.ga8...@jwilk.net
