[Pkg-graphite-maint] ITP carbonate

Andreas Rütten AndreasRuetten at gmx.de
Sun Oct 5 13:59:42 UTC 2014


Hi Mathieu,

Am Sat, 4 Oct 2014 21:44:23 +0200
schrieb Mathieu Parent <math.parent at gmail.com>:

> 2014-10-04 18:38 GMT+02:00 Andreas Rütten <AndreasRuetten at gmx.de>:
> [...]
> > In the last weeks I did make some progress on the packaging stuff
> > and it would be much appreciated if somebody of you could take a
> > look at the current state if there is something missing before the
> > package could be sponsored.
> >
> > The git repo is there:
> > http://anonscm.debian.org/cgit/pkg-graphite/packages/carbonate.git/
> 
> I did a quick review.
> 
> Some notes:
> - from build log "Ran 0 tests in 0.000s"
> => it would be good to enable tests. At least using dep8

Yeah, I also stumbled across this. I tried half the day to establish
some test during buildtime yesterday. I tried using the provided
tox.ini from upstream and also just calling the test files with python
unittest. But in both cases with not so much success. If you are
interested I have both attempts in separate branches in my local git
repo and could push them to alioth or provide you the patches as files.

BTW: Are you meaning dep8 from debian or pep8 from python?

> - from build log "I: dh_python2 pydist:181: Cannot find installed
> package that provides whisper. Using python-whisper as package name.
> Please add "whisper correct_package_name" line to
> debian/py3dist-overrides to override it IF this is incorrect."
> => maybe a buildep on python-whisper?

It should be no buildep but it's declared as install_requires in the
setup.py. I added a debian/pydist-overrides for whisper and the warning
is gone.


Cheers,
Andreas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-graphite-maint/attachments/20141005/9e29a94c/attachment.sig>


More information about the Pkg-graphite-maint mailing list