Bug#671403: RFS: python-lzo/1.08-1 [ITP] -- Python bindings for the LZO data compression library

[Jakub Wilk <jwilk@debian.org>]

* Mehdi Abaakouk <sileht@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