Bug#802536: RFS: python-ncclient/0.4.5-1

2015-11-01 Thread Sebastien Badia 
On Thu, Oct 22, 2015 at 11:18:08AM (+0200), Thomas Goirand wrote:
> Hi Seb,
> 
> A quick review then.
> 
> Both Ubuntu LTS and Debian Stable have version >= 3.3.1 of python-lxml,
> so you don't need to specify the version in debian/control.
> 
> Same remark for paramiko.
> 
> Your package doesn't run any of the upstream tests. Why?
> 
> Both -lxml and -paramiko could be in build-depends-indep.
> 
> In the doc's conf.py, there's "sphinx.ext.intersphinx". This is to be
> patched out, so that the package doesn't do network access during build.

Hi! Thanks zigo for the review,

Just fixed the package according your valuable comments, if you have a bit of
time in order to re-take a look it would be cool :-)

Thanks in advance!

https://tracker.debian.org/pkg/python-ncclient
http://mentors.debian.net/package/python-ncclient
http://mentors.debian.net/debian/pool/main/p/python-ncclient/python-ncclient_0.4.5-1.dsc

Seb


signature.asc
Description: PGP signature

Bug#802536: RFS: python-ncclient/0.4.5-1

2015-10-22 Thread Thomas Goirand 
On 10/20/2015 10:24 PM, Sebastien Badia wrote:
> Package: sponsorship-requests
> Severity: normal
> 
> Dear mentors,
> 
> I am looking for a sponsor for my package "python-ncclient"
> 
> * Package name: python-ncclient
>   Version : 0.4.5-1
>   Upstream Author : Leonidas Poulopoulos 
> * URL : http://ncclient.org/
> * License : Apache-2
>   Section : python
> 
> It builds those binary packages:
> 
>   python-ncclient - Python library for NETCONF clients
> 
> To access further information about this package, please visit the following 
> URL:
> 
>  http://mentors.debian.net/package/python-ncclient
> 
> 
> Alternatively, one can download the package with dget using this command:
> 
>   dget -x 
> http://mentors.debian.net/debian/pool/main/p/python-ncclient/python-ncclient_0.4.5-1.dsc
> 
> Note: This upload introduce the new upstream version 0.4.5 (pre python3.x).
> 
> Thanks in advance, cheers,
> 
> Seb

Hi Seb,

A quick review then.

Both Ubuntu LTS and Debian Stable have version >= 3.3.1 of python-lxml,
so you don't need to specify the version in debian/control.

Same remark for paramiko.

Your package doesn't run any of the upstream tests. Why?

Both -lxml and -paramiko could be in build-depends-indep.

In the doc's conf.py, there's "sphinx.ext.intersphinx". This is to be
patched out, so that the package doesn't do network access during build.

Cheers,

Thomas Goirand (zigo)

Bug#802536: RFS: python-ncclient/0.4.5-1

2015-10-20 Thread Sebastien Badia 
Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package "python-ncclient"

* Package name: python-ncclient
  Version : 0.4.5-1
  Upstream Author : Leonidas Poulopoulos 
* URL : http://ncclient.org/
* License : Apache-2
  Section : python

It builds those binary packages:

  python-ncclient - Python library for NETCONF clients

To access further information about this package, please visit the following 
URL:

 http://mentors.debian.net/package/python-ncclient


Alternatively, one can download the package with dget using this command:

  dget -x 
http://mentors.debian.net/debian/pool/main/p/python-ncclient/python-ncclient_0.4.5-1.dsc

Note: This upload introduce the new upstream version 0.4.5 (pre python3.x).

Thanks in advance, cheers,

Seb

-- 
Sebastien Badia


signature.asc
Description: PGP signature