RFS: python-whisper

2012-01-22 Thread Jonas Genannt
Hello,

I'm searching for an sponsor for the package python-whisper.

Package is located at svn:


http://anonscm.debian.org/viewvc/python-modules/packages/python-whisper/trunk/debian/


Could any DD please check my package, and if it fits upload it?

Thanks,
Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120122124537.67af8...@swetlana.brachium-system.net



Re: RFS: python-whisper

2012-01-27 Thread Jonas Genannt
Hello,

I'm searching for an sponsor for the package python-whisper.

Package is lintian clean and already commited to the SVN:

http://anonscm.debian.org/viewvc/python-modules/packages/python-whisper/trunk/debian/

Could any DD please upload my package?

Thanks,
Jonas


On Sun, 22 Jan 2012 12:45:37 +0100
Jonas Genannt  wrote:
> Hello,
> 
> I'm searching for an sponsor for the package python-whisper.
> 
> Package is located at svn:
> 
>   
> http://anonscm.debian.org/viewvc/python-modules/packages/python-whisper/trunk/debian/
> 
> 
> Could any DD please check my package, and if it fits upload it?
> 
> Thanks,
>   Jonas
> 
> 


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120127091726.0e181b0a@swetlana.internetstores.local



Re: RFS: python-whisper

2012-01-28 Thread Jonas Genannt
Hello,

thanks for checking the package.

> Do you know why upstream doesn't include text of the Apache license
> in their tarball anymore?

I don't know but PKG-INFO and whisper.py holds information about Apache
license.

> >  * d/control: Added myself as Uploader / changed Module Team to
> > Maintainer
> 
> Was this change acknowledged by Elliot?

yes, please see #651216.

 
> >  * d/control: DM-Upload-Allowed: yes
> 
> Uh-uh. Without this change please.

reverted and commit into svn

> 
> >  * d/copyright: changed to DEP5
> 
> If you enable pedantic tags, lintian emits:
> P: python-whisper source: out-of-date-copyright-format-uri
> http://anonscm.debian.org/viewvc/dep/web/deps/dep5.mdwn?revision=135

yes I know, but when I use http://dep.debian.net/deps/dep5/ lintian
tells me:

P: python-whisper source: unversioned-copyright-format-uri
http://dep.debian.net/deps/dep

N:Please use
N:http://anonscm.debian.org/viewvc/dep/web/deps/dep5.mdwn?revision= as the format URI instead.

But this urls is wrong, it 404.

I have changed it into:

http://anonscm.debian.org/viewvc/dep/web/deps/dep5/copyright-format.xml?revision=248

But now lintian complains about unknown-copyright-format-uri ;)

Greets,
Jonas




-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120128144523.76903...@swetlana.brachium-system.net



Re: RFS: python-whisper

2012-01-28 Thread Jonas Genannt
Hello,

> >http://anonscm.debian.org/viewvc/dep/web/deps/dep5/copyright-format.xml?revision=248
> >
> >But now lintian complains about unknown-copyright-format-uri ;)
> 
> Yes, DEP-5 and its format URIs is a mess. I pointed out this lintian
> tag only because the specification has been overhauled since r135.

yes you are right. :)

> 
> I made some tiny modifications to debian/changelog (I hope you don't 
> mind!) and uploaded the package. Thanks!

no problem, Thanks for your review and your upload!

Greets,
Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120128195354.11313...@swetlana.brachium-system.net



RFS: python-carbon / graphite

2012-02-04 Thread Jonas Genannt
Hello,

I'm searching an sponsor for the python-carbon package. This package is
required for Graphite (http://graphite.wikidot.com/).

ITP is field: #651233

The package is located at svn:


http://anonscm.debian.org/viewvc/python-modules/packages/python-carbon/trunk/debian/

Could any DD please check and upload the package?

Thanks,
Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120204180800.06eef...@swetlana.brachium-system.net



Re: RFS: python-carbon / graphite

2012-02-04 Thread Jonas Genannt
Hello,

> > I'm searching an sponsor for the python-carbon package. This
> > package is required for Graphite (http://graphite.wikidot.com/).
> 
> Since i'm interested in graphite, here's a quick review:

thanks for your review.

> 
> general
> - package name: is it more a module or a program (ie, how a general
> user is supposed to use the package)? if the former, the package name
> it's ok, else it should be changed (and moved to PAPT).

I named carbon package python-carbon because of the existing
python-whisper package.

Perhaps the packages should be named:

graphite-whisper
graphite-carbon
graphite-web(the webinterface, not yet packaged)

> - there are some lintian warnings: how have you addressed them?

This are only warnings about missing manpages.

> - the package install a plugin directly into twisted dir: are you sure
> this is the correct way to do that?

Seems to be normal - perhaps ask twisted maintainers.

> 
> debian/control
> - add Vcs-* fields

done

> - missing X-Python-Version field

done

> - improve long description: knowing the server listens on port 2003
> (and stuff like that) it's not the purpose of the description

will do it.

> 
> debian/rules
> - why all those file/directory tests?

removed
> 
> debian/README.Debian
> - small grammar fixes: '...to configure carbon, copy..."; s/Than/Then

done

> 
> debian/python-carbon.examples
> - why not installing the configuration files in the right directory,
> /etc, instead of as examples the user has to copy by-hand? in any
> case, the user has to change default settings to start the server.

yes, I can do that.

> 
> debian/python-carbon.dirs
> - why configuring dir in a namespace not for python-carbon (namely
> /var/lib/graphite*)?

see next.

> debian/python-carbon.postinst
> - why is this script taking care of creating "Graphite User"

because the whisper database files should owned by the user that runs
the webinterface. If you have different users you will get troubles
running the webinterface and carbon is "part" of Graphite. Without
Carbon you can't run Graphite. We can name that directory carbon but
this will confuse users.

Thanks,
Jonas







-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120204203640.2145d...@swetlana.brachium-system.net



Re: RFS: python-carbon / graphite

2012-02-04 Thread Jonas Genannt
Hello Vincent,

> > Could any DD please check and upload the package?
> 
> After Sandro remark about Twisted, I have  looked a bit at it too:

thanks for checking the package.

> is to use Debian-graphite instead.  The other one is to  use an
> underscore (_graphite).

done. Changed username to _graphite.

> 
>  2. I don't see the point to check if graphite UID/GID is 0.

removed.

> 
>  3. For  the Twisted  plugin,  you need  to  have a  triggers file
> with "activate  twisted-plugins-cache"  to let  the  plugin  cache
> to  be rebuilt.

thanks, I have added that.

> 
>  4. I  think that there  are some actions  missing in postrm in  case
> of purge.  For  example,  the  content  of  /var/log/carbon  should
> be deleted. I suppose that this also applies to some other
> directories.

I have added an postrm to delete /var/log/carbon. /var/lib/graphite
should not be deleted on purge, because is contains userdata. 

Perhaps we should an an debconf question on install "delete database
directory on purge" and defaults into not delete the files.

 
>  5. You  should cleanup the init  script. It contains  comments aimed
> at building it. Moreover, you  should check for VERBOSE variable
> before using echo.

you are right. cleaned the file.


I have commited my changed into svn.

Thanks,
Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120205011451.7014c...@swetlana.brachium-system.net



Re: RFS: python-carbon / graphite

2012-02-05 Thread Jonas Genannt
Hello,

> > Perhaps we should an an debconf question on install "delete database
> > directory on purge" and defaults into not delete the files.
> 
> The policy  does not mandate anything on  this area. Feel free  to
> add a debconf question  if you  want. If  you add a  debconf
> question,  try to steal the question  from some other packages. I
> think that mysql-server has such a  question (or maybe this is
> dbconfig-common). This will help translators.

I have added an debconf question on postrm. It's commit into svn.

I have also changed the long description of python-carbon.

Now I will change the package name from python-carbon into
graphite-carbon and move it to PAPT. (already sent an request to add me
to PAPT)

Is there any other problems with the package that will deny uploading?

Thanks,
Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120205125437.75051...@swetlana.brachium-system.net



Re: RFS: graphite-carbon

2012-02-06 Thread Jonas Genannt
Hello,

> I'm searching an sponsor for the python-carbon package. This package
> is required for Graphite (http://graphite.wikidot.com/).
> 
> ITP is field: #651233

I have renamed the carbon package into graphite-carbon and moved it
into Python App PKG. 

http://anonscm.debian.org/viewvc/python-apps/packages/graphite-carbon/trunk/debian/

Could any DD please check and upload the package if it fits?

Thanks,
Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120206202557.77f8b...@swetlana.brachium-system.net



RFS: graphite-carbon

2012-02-12 Thread Jonas Genannt
Hello,

I have packaged the carbon from graphite. (ITP is field: #651233)

The package is lintian clean. 

http://anonscm.debian.org/viewvc/python-apps/packages/graphite-carbon/trunk/debian/

Could any DD please check and upload the package if it fits?

Thanks,
Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120212154223.26f37...@swetlana.brachium-system.net



Re: RFS: graphite-carbon

2012-02-15 Thread Jonas Genannt
Hello,

> I have packaged the carbon from graphite. (ITP is field: #651233)
> 
> The package is lintian clean. 
> 
> http://anonscm.debian.org/viewvc/python-apps/packages/graphite-carbon/trunk/debian/

could please any DD have a look at my package - and if it fits upload
it into NEW?

Thanks,
Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120216082426.153a4ebf@swetlana.internetstores.local



RFS: graphite-carbon

2012-03-12 Thread Jonas Genannt
Hello,

I have packaged the carbon daemon from graphite. (ITP is field: #651233)

http://anonscm.debian.org/viewvc/python-apps/packages/graphite-carbon/trunk/debian/

Could any DD please check and upload the package if it fits?

Thanks,
Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120312210236.34d7c...@swetlana.brachium-system.net



Re: RFS: graphite-carbon

2012-03-19 Thread Jonas Genannt

> I have packaged the carbon daemon from graphite. (ITP is field:
> #651233)
> 
> http://anonscm.debian.org/viewvc/python-apps/packages/graphite-carbon/trunk/debian/
> 
> Could any DD please check and upload the package if it fits?

is there any change to get that package into debian?!?

Thanks,
Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120319105230.289f21bc@swetlana.internetstores.local



RFS: graphite-carbon

2012-03-31 Thread Jonas Genannt
Hello,

I have packaged the carbon daemon from graphite. (ITP is field: #651233)

http://anonscm.debian.org/viewvc/python-apps/packages/graphite-carbon/trunk/debian/

Could any DD please check and upload the package if it fits?

Thanks,
Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120331152507.73014...@swetlana.brachium-system.net



RFS: python-whisper (0.9.10)

2012-06-02 Thread Jonas Genannt
Hello,

I have updated python-whisper to version 0.9.10. Package is lintian
clean.

Package is available via svn:

http://anonscm.debian.org/viewvc/python-modules/packages/python-whisper/trunk/

Could any DD please upload the new package into Debian?

Thanks,
Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120602133621.2af35...@swetlana.brachium-system.net



Re: RFS: python-whisper (0.9.10)

2012-06-02 Thread Jonas Genannt
Hallo,

thanks for your review.

> * Jonas Genannt , 2012-06-02, 13:36:
> >http://anonscm.debian.org/viewvc/python-modules/packages/python-whisper/trunk/
> 
> Typo in whisper-dump.py: reda -> read.

=> fixed and submitted and patch to upstream.

> 
> Typo in debian/whisper-dump.1: whister -> whisper.

=> fixed

> 
> In whisper.py, "#!/usr/bin/env python" has been moved to last line of 
> the file. This is very odd. Is that intentional?

since whisper.py is only an library it should not hurt us. If you wish,
I can create an patch against that problem.

> 
> "rename" is provided by perl, so you should build-depend on it. (perl 
> happens to be transitively build-essential, since dpkg-dev depends on 
> it. But you shouldn't rely on that.) I would also tighten the regex
> by adding a $ at the end of it.
> 

thanks, added the build dependency of perl. I think I don't need the $
at the regex, because I selected only *.py on the filepath.

Thanks,
Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120602200936.6ac8a...@swetlana.brachium-system.net



Re: RFS: python-whisper (0.9.10)

2012-06-02 Thread Jonas Genannt
Hello,

> In whisper-dump, lines 27 and 36, we have:
> 
>   raise CorruptWhisperFile("...")
> 
> But no such exception is defined or imported. Perhaps it was meant to
> be whisper.CorruptWhisperFile? But whisper.CorruptWhisperFile
> constructor takes two arguments, not one.

I will submit an bug into upstream BTS. whisper-dump is not so
important for whisper library.

Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120602214354.1bad2...@swetlana.brachium-system.net



Re: RFS: python-whisper (0.9.10)

2012-06-04 Thread Jonas Genannt
Hello,

> >whisper-dump is not so important for whisper library.
> 
> Fair enough. Uploaded.

thanks!

Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20120604114327.15051f0b@swetlana.internetstores.local



RFS: python-whisper (0.9.12)

2013-08-27 Thread Jonas Genannt
Hello,

I have updated python-whisper to version 0.9.12. 

Package is available via svn:

http://anonscm.debian.org/viewvc/python-modules/packages/python-whisper/trunk/

Could any DD please review and if it fits upload the new package into Debian?

Thanks,
Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20130827225631.4b111...@swetlana.brachium-system.net



RFS: python-pypuppetdb -- interface to the PuppetDB REST API

2014-09-11 Thread Jonas Genannt
Hello,

could any dd please have a look on python-pypuppetdb? - It's an interface for 
PuppetDB,
it's required for Puppetboard. 

 pypuppetdb (0.1.1+git080614-1) unstable; urgency=low
 .
   * Initial release. (Closes: #756009)


It's available in the new Git:

git+ssh://git.debian.org/git/python-modules/packages/pypuppetdb.git


Thanks!
Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/20140911205306.7de6d...@swetlana.brachium-system.net



Re: RFS: python-pypuppetdb -- interface to the PuppetDB REST API

2014-09-11 Thread Jonas Genannt
Hello,

> > could any dd please have a look on python-pypuppetdb? - It's an
> > interface for PuppetDB, it's required for Puppetboard.
> >
> >  pypuppetdb (0.1.1+git080614-1) unstable; urgency=low
> >  .
> >* Initial release. (Closes: #756009)
> >
> >
> > It's available in the new Git:
> >
> > git+ssh://git.debian.org/git/python-modules/packages/pypuppetdb.git
> >
> >
> 
> Disclaimer: I am not a DD so I can not sponsor your work.

thanks for having a lock.
> 
> 1) I see that you have package the head of master instead of a
> released version. I wouldn't recommend this since it is difficult to
> manage. For example, this confuses uscan:
> 
> $ uscan --report
> Processing watchfile line for package pypuppetdb...
> Newest version on remote site is 0.1.1, local version is
> 0.1.1+git080614 pypuppetdb: remote site does not even have current
> version
> 
> If you feel that the more recent commits are really worth having,
> encourage upstream to make a new release.

I need that version and not striped it, because upsream will release
soon a new version. I need that git release because upstream was
missing a copyright statement. Ăšpstream has accepted my Pull Requests
but not released a new version. That's why we have a master-release.


> 
> 2) In d/rules you have:
> 
> # disable tests, require network access
> override_dh_auto_test:
> 
> I had a quick glance and it appeared that the http requests were being
> properly mocked to not require the network. If there are a couple of
> tests that require the network, it's better to not run those or patch
> them out than skip the suite entirely. Bonus points for patching the
> tests to mock the requests and submitting the patch upstream/

I had a try once, but I don't figure out the tox stuff. I had added
once python-requests,  python3-requests, python-tox.

But running the tests:

I: pybuild base:170: python2.7 setup.py test 
running test
Searching for tox
Reading https://pypi.python.org/simple/tox/
Download error on https://pypi.python.org/simple/tox/: [Errno 111]
Connection refused -- Some packages may not be found! Couldn't find
index page for 'tox' (maybe misspelled?) Scanning index of all packages
(this may take a while) Reading https://pypi.python.org/simple/
Download error on https://pypi.python.org/simple/: [Errno 111]
Connection refused -- Some packages may not be found! No local packages
or download links found for tox error: Could not find suitable
distribution for Requirement.parse('tox') E: pybuild pybuild:256: test:
plugin distutils failed with: exit code=1: python2.7 setup.py test 

Do you have a idea how to run these?


> 3) debian/python-pypuppetdb.debhelper.log
> python3-pypuppetdb.debhelper.log
> 
> Remove these files.

I don't see that files in the git repo. So can't removed.



Thanks,
Jonas


--
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/20140912081804.500b4e7e@branka.internetstores.local



Re: RFS: python-pypuppetdb -- interface to the PuppetDB REST API

2014-09-22 Thread Jonas Genannt
Hello,

> >Download error on https://pypi.python.org/simple/tox/: [Errno 111]
> >Connection refused -- Some packages may not be found! Couldn't find
> >index page for 'tox' (maybe misspelled?) Scanning index of all packages
> >(this may take a while) Reading https://pypi.python.org/simple/
> >Download error on https://pypi.python.org/simple/: [Errno 111]
> >Connection refused -- Some packages may not be found! No local packages
> >or download links found for tox error: Could not find suitable
> >distribution for Requirement.parse('tox') E: pybuild pybuild:256: test:
> >plugin distutils failed with: exit code=1: python2.7 setup.py test 
> 
> pybuild automatically sets http_proxy and https_proxy to the discard port,
> disabling all network access.  This is generally a good thing because most
> buildds won't let you access the internet either, so you want these to fail on
> local builds.
> 
> When you see the error above, it generally means there's a missing build
> dependency.

I tried to get the unit tests running, but I can't. The library itself works 
great, I'm
using it with Puppetboard.

So can we upload python-pypuppetdb without the tests?

Thanks,
Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/20140922202720.7b5f7...@swetlana.brachium-system.net



Re: RFS: python-pypuppetdb -- interface to the PuppetDB REST API

2014-10-07 Thread Jonas Genannt
Hello,

> pybuild automatically sets http_proxy and https_proxy to the discard
> port, disabling all network access.  This is generally a good thing
> because most buildds won't let you access the internet either, so you
> want these to fail on local builds.
> 
> When you see the error above, it generally means there's a missing
> build dependency.

git+ssh://git.debian.org/git/python-modules/packages/pypuppetdb.git 


I have now enabled the unit tests on build time. Could any DD please
have a look and upload it?

Thanks,
Jonas


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/20141007153406.00b6b01c@branka.internetstores.local