Re: RFS: python-unicodecsv (Closes: #669678)

[Thomas Bechtold <thomasbechtold@jpberlin.de>]

On Mon, 2012-04-23 at 16:49 +0200, Jakub Wilk wrote:

i fixed most of the comments and uploaded a new version to
mentors.debian.net . see comments below

> "debhelper (>= 7.0.50)" - I think this could be simply "(>= 7)", I don't 
> see anything in your package that would require 7.0.50.

fixed in debian/control

> s/unicode/Unicode/ in the package description.

fixed in debian/control

> Please run tests at build time.

fixed in debian/rules

> The package fails to build:
> | dh clean --with python2
> |    dh_testdir
> |    dh_auto_clean
> | Traceback (most recent call last):
> |   File "setup.py", line 3, in <module>
> |     from setuptools import setup, find_packages
> | ImportError: No module named setuptools
> | dh_auto_clean: python setup.py clean -a returned exit code 1

fixed in debian/control build-dependencies

> Lintian reports:
> I: python-unicodecsv source: debian-watch-file-is-missing

fixed in debian/watch

> I: python-unicodecsv: extended-description-is-probably-too-short

i don't know what i should write about the package. sorry.


Cheers,

Tom