Hello Jakub,
Jakub Wilk [2012-11-03 16:30 +0100]:
> Yup, please feel free.
Thanks. Scott Kitterman confirmed this as well last week.
> I had a quick look at the package. Unfortunately, it FTBFS:
> | python3 setup.py build
> | Traceback (most recent call last):
> | File "setup.py", line 13, in <module>
> | with open('NEWS') as f:
> | IOError: [Errno 2] No such file or directory: 'NEWS'
Known, and fixed in upstream trunk (forgot to dist the file). I'll do
a new upstream release ASAP, I just ran out of time yesterday.
FTR, I committed the current packaging to SVN already. That was copied
straight out of my Ubuntu PPA packaging branch, but as that has full
source I didn't notice the lack of NEWS before.
> Please add ${python3:Depends} to Depends.
>
> In debian/rules you run "setup.py install" with all supported Python
> 3 versions, but "setup.py build" and "setup.py test" only with the
> default one. This looks weird.
Indeed, it would be cleaner to build/test for all Pythons. I wish
debhelper would support py3 properly, to avoid propagating those
errors by copy&paste'ing rules files from other packages.. I'll get
that fixed before upload as well, thanks for spotting!
Martin
--
Martin Pitt | http://www.piware.de
Ubuntu Developer (www.ubuntu.com) | Debian Developer (www.debian.org)
--
To UNSUBSCRIBE, email to debian-wnpp-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20121103155943.gh3...@piware.de
