Re: FYI: Fwd: RFS: pygithub/1.35-1[ITA]

2018-01-17 Thread Filip Pytloun
Hello Emmanuel,

few notes to your package:

- it would be good to move under DPMT (have you joined the team
  already?), create git repository and use git-dpm to import current
  version and make your changes, also set Maintainer and Vcs-* to
  reflect this

- You should update bug #851187 and change topic s,RFA,ITA + assign to
  yourself

- d/compat + d/control: update debhelper to latest 11

- d/copyright:
- add yourself into copyright holders for debian/*
- use https in Format url

- d/control:
- X-Python-Version and X-Python3-Version are not needed
- Add support for autopkgtest (Testsuite: autopkgtest-pkg-python)
- Run `wrap-and-sort -bast` if you haven't already

Filip

On 2018/01/17 21:37, eamanu15 . wrote:
> FYI
> 
> -- Forwarded message -
> From: eamanu15 . 
> Date: mié., 17 de ene. de 2018 a la(s) 18:37
> Subject: RFS: pygithub/1.35-1[ITA]
> To: Debian Mentors List 
> 
> 
> Package: sponsorship-requests
> Severity: normal
> 
> Dear mentors,
> 
> I am looking for a sponsor for my package "pygithub"
> 
> * Package name: pygithub
>   Version : 1.35-1
>   Upstream Author : Adam Dangoor 
> Vincent Jacques 
> Jeremy Phelps 
> * URL : https://pypi.python.org/pypi/PyGithub
> * License : LGPL-3+
>   Section : python
> 
> It builds those binary packages:
> python-github - Access to full Github API v3 from Python2
> python3-github - Access the full Github API v3 from Python3
> 
> To access further information about this package, please visit the
> following URL:
> https://mentors.debian.net/package/pygithub
> 
> Alternatively, one can download the package with dget using this command:
> dget -x
> https://mentors.debian.net/debian/pool/main/p/pygithub/pygithub_1.35-1.dsc
> 
> Changes since the last upload:
>* New upstream release
>* Update Maintainer field from debian/control to intent to adopt
> the package (Closes: #851187)
>* Update Standards-Version from  3.9.6 to 4.1.3 from debian control
> according to the up-to-date standards
>* Add new maintainer on copyright debian/* files field from
> debian/copyright to stay up-to-date
>* Remove override_dh_installchangelogs target from debian/rules due to
> the new version (1.35) do not have README.rst file
> 
> Regards,
> Emmanuel
> 
> 
> -- 
> Arias Emmanuel
> https://www.linkedin.com/in/emmanuel-arias-437a6a8a
> http://eamanu.com
> 
> 
> -- 
> Arias Emmanuel
> https://www.linkedin.com/in/emmanuel-arias-437a6a8a
> http://eamanu.com


signature.asc
Description: PGP signature


Re: Wrong package name python-jwt

2016-08-29 Thread Filip Pytloun
Hello Cornelius,

Pretty much on this topic is explained in dh_python2 man page (find
dependencies)
http://manpages.ubuntu.com/manpages/precise/man1/dh_python2.1.html

And here's a little bit more (but I suppose you already know this doc):
https://wiki.debian.org/Python/LibraryStyleGuide

Filip

On 2016/08/29 13:04, Cornelius Kölbel wrote:
> Hello Piotr,
> 
> thanks a lot. That did the trick.
> Please enlighten me. dh-python maps python module names to debian
> package names?
> 
> Kind regards
> Cornelius
> 
> Am Montag, den 29.08.2016, 12:47 +0200 schrieb Piotr Ożarowski:
> > [Cornelius Kölbel, 2016-08-29]
> > > 
> > > Due to ${python:Depends} the dependency python-pyjwt is added.
> > please add dh-python to Build-Depends and try again
> -- 
> Cornelius Kölbel
> cornelius.koel...@netknights.it
> +49 151 2960 1417
> 
> NetKnights GmbH
> http://www.netknights.it
> Landgraf-Karl-Str. 19, 34131 Kassel, Germany
> Tel: +49 561 3166797, Fax: +49 561 3166798
> 
> Amtsgericht Kassel, HRB 16405
> Geschäftsführer: Cornelius Kölbel
> 




signature.asc
Description: Digital signature


PAPT membership request

2016-08-17 Thread Filip Pytloun
Hello,

can someone please approve my PAPT membership (already requested on
Alioth)?

Docs states that:

"Any Python Modules Packaging Team member who wishes to integrate his
packages in the team can do so without requesting access"

but I get permission denied when creating SVN repository for new package
(vdirsyncer) because I am not in python-apps group.

Maybe I don't understand correctly the "integrate packages in the team"
or that statement doesn't seem to be true?

Thank you,
Filip


signature.asc
Description: Digital signature


Re: DPMT membership request

2016-08-08 Thread Filip Pytloun
Hello Zigo,

thank you very much for your feedback!
More inline:

> 1/ debian/rules
> ===
> 1.a. Simplifying dpkg-parsechangelog
> 
> You can replace your PKG = line with:
> 
> PKG = $(shell dpkg-parsechangelog -SSource)
> 
> The same way, you can simplify VER = using dpkg-parsechangelog -SVersion
> 
> Note that I'm not adding a space after the -S, as this would add an
> incompatibility with Ubuntu trusty (and there's no reason why you
> wouldn't care as this is a gratuitous fix for it).

I found this at https://wiki.debian.org/onlyjob/get-orig-source
Didn't know these shortcuts anyway, fixing.

> 
> 1.b. Adding unnecessary parenthesis
> 
> Why are you adding parenthesis to the command:
> PYTHONPATH=. help2man

Yes, that should not be needed, removing.

> 
> ?
> 
> 1.c. Using update-alternatives for /usr/bin
> 
> Instead of doing:
>   rm -rf debian/python-aptly/usr/bin
>   mkdir -p debian/aptly-publisher/usr
>   mv debian/python3-aptly/usr/bin debian/aptly-publisher/usr/
> 
> you can have a look at using pkgos-alternative-bin from
> openstack-pkg-tools. This will generate postinst, prerm and postrm for
> you, so it handles /usr/bin with python2 or python3 alternatives.
> Optionally, you can fix py3 as higher priority in the postinst files if
> you prefer.

I was thinking about using alternatives vs forcing python3 (didn't know
pkgos-alternative-bin anyway or any reference package for this case) but
decided to force Python 3 as it should be Debian's default.

> 
> 2/ debian/control
> =
> 2.a X-Python{3,}-Version headers
> 
> Please remove X-Python-Version: >= 2.6 and X-Python3-Version: >= 3.2
> stuff. Jessie has Python 2.6 and Python 3.4, so these are completely
> useless. It isn't even needed in oldstable!!!

Good, didn't know it's not needed now.

> 
> 2.b Build-Depends-Indep:
> 
> Since both all of the generated binaries are Arch: all, you can put some
> of the Build-Depends into Build-Depends-Indep: like this:
> 
> Build-Depends: debhelper (>= 9),
>dh-python,
>help2man,
>python-all,
>python-setuptools,
>python3-all,
>python3-setuptools
> Build-Depends-Indep: python-requests,
>  python-yaml,
> 
> I was also very much surprised to not see python3-requests and
> python3-yaml in your build-depends. Aren't you running Py3 tests too?

Will fix this.

> 
> 2.c Short and long description
> 
> Your short description for python-aptly and python3-aptly shouldn't be
> that different. Also, upstream short desc is nicer. I'd suggest:
> 
> Package: python-aptly
> Description: REST API client  and tooling - Python 2.7
> [...]
> Package: python3-aptly
> Description: REST API client  and tooling - Python 3.x
> 
> 2.d Long desc for Package: aptly-publisher
> 
> The long desc start by an empty line. This isn't nice. Also, please
> consider longer long descriptions. 2 lines isn't enough, at least 5
> would be nicer. Upstream has a lot more info which you can pickup.

Fixed.

> 
> 3/ Final word
> 
> I hope this helps. Thanks for your contribution to Debian.

Helps a lot, thank you again!

> 
> Cheers,
> 
> Thomas Goirand (zigo)
> 


signature.asc
Description: Digital signature


Re: DPMT membership request

2016-08-05 Thread Filip Pytloun
Thank you very much for your interest, my sponsor/mentor is already
Ondrej Novy :-)

I have a few more tools that I authored and may be interesting to
package (any opinion appreciated):

EC2 security groups manager: https://github.com/gooddata/sgmanager
 - supports only EC2 API so it can be used on OpenStack but a shame it
   doesn't support neutron API at the moment, don't know if there will
   be any interest in this one :-X

Smoke testing framework: https://github.com/gooddata/smoker
 - not only for smoke testing but for executing anything over set of
   nodes and gathering unified result, can be interesting for some
   people

Smart files/directory cleaner: https://github.com/gooddata/tmpcleaner
 - I think this one is very useful especially on slow shared filesystems
   (eg. GlusterFS) where passing deep directory structure is expensive
   and takes a lot of time (eg. hours in environment for which this was
   developed). I don't know about any alternative tool working as this
   one.

On 2016/08/04 18:10, Sandro Tosi wrote:
> On Thu, Aug 4, 2016 at 4:35 PM, Filip Pytloun  wrote:
> > As the first package, I am working on packaging of python-aptly:
> > https://github.com/fpytloun/debian-python-aptly
> 
> this package looks interesting, let me know if you need a sponsor and
> I'd be happy to be one
> 
> -- 
> Sandro "morph" Tosi
> My website: http://sandrotosi.me/
> Me at Debian: http://wiki.debian.org/SandroTosi
> G+: https://plus.google.com/u/0/+SandroTosi


signature.asc
Description: Digital signature


DPMT membership request

2016-08-04 Thread Filip Pytloun
Hello DPMT,

I would like to join to be able to package some software I am interested
in (Openstack-related + various python tools that I (co-)authored).

As the first package, I am working on packaging of python-aptly:
https://github.com/fpytloun/debian-python-aptly

I have read and agree the policy.
My Alioth login is fpytloun-guest.

Thank you,
Filip


signature.asc
Description: Digital signature