On Tue, Jan 20, 2015 at 9:01 PM, Sebastiaan Couwenberg <sebas...@xs4all.nl> wrote: > Hi Johan, > > Sorry for not doing this sooner, but I have reviewed the package now. > Some comments follow. Thanks! > > The copyright file only documents the upstream copyright, documenting > the copyright & license for debian/* is a good idea unless you want to > assign the copyright to MapBox.
Good catch > > There seems to be an issue with the setup.py and Python 3.2, the package > fails to build because the clean target fails before even starting the > build: > > This is on one of my wheezy systems, the package builds fine with Python > 3.4 on sid. On jessie it works as well. I don't think it is useful to change it. > > lintian reports a duplicate short description. Because the short > description is already 60 characters long this doesn't leave much room > to add something like "Python 3 module for ...". Appending "for Python > V" would fall within 80 characters, although the line length including > field name exceeds it. Improved the short description > > Kind Regards, > > Bas > > -- > GPG Key ID: 4096R/E88D4AF1 > Fingerprint: 8182 DE41 7056 408D 6146 50D1 6750 F10A E88D 4AF1 > -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/cajop35mjt6boq+e_ckynahtr9guu0u-bj9zdbgfmaoo8ari...@mail.gmail.com
