Path: csiph.com!aioe.org!gothmog.csi.it!bofh.it!news.nic.it!robomod
From: Tiago Ilieve <tiago.myhro@gmail.com>
Newsgroups: linux.debian.maint.python
Subject: Re: Packaging pythonpy
Date: Thu, 31 Mar 2016 02:30:02 +0200
Message-ID: <riyEO-4UC-15@gated-at.bofh.it>
References: <rgHjc-7IB-7@gated-at.bofh.it> <rgIoV-8rq-3@gated-at.bofh.it> <rgK7n-1az-1@gated-at.bofh.it> <ri9Gq-3IQ-21@gated-at.bofh.it> <rijZ9-2KA-19@gated-at.bofh.it> <riuhQ-1Jv-9@gated-at.bofh.it>
Old-Return-Path: <tiago.myhro@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Approved: robomod@news.nic.it
Lines: 65
Organization: linux.* mail to news gateway
Sender: robomod@news.nic.it
Xref: csiph.com linux.debian.maint.python:8366

Dmitry,

On 30 March 2016 at 16:48, Dmitry Shachnev <mitya57@debian.org> wrote:
> Looks like I was a bit mistaken =E2=80=94 dh_python2 will not replace she=
bangs for
> files in /usr/share. But then you can do this manually using a sed call [=
1],
> that is still easier than a patch.

This is indeed way clever than an entire patch to fix something so simple. =
:-)

Done[1].

> Re Lintian error, this looks like a false positive. See [2].

You mean that maybe it's better if we add an override instead of a workarou=
nd?

> Minor nit about your package: please build-depend on dh-python to get a m=
odern
> version of dh_python2.

Done[2].

> Major nit about your package: did you read the Python policy, in particul=
ar
> the part that tells that all new packages should use Python 3 [3]?

Actually I had consulted it, but not read it entirely. I don't know
why but I thought that the Python 3 requirement was a *nice to have*,
not a *should have*. Anyway, I've updated[3] the build system to use
Python 3.

I noticed that the test suite wasn't being properly executed and sent
a patch to upstream[4]. As soon as a new release is made with this
changed integrated, I'll be adding support for DEP-8 (as suggested by
Barry Warsaw[5] in the last week), as they are functional tests that
depends on the package being installed.

I've uploaded the updated package to mentors.d.n[6], but I guess its
better to wait for a new release integrating the test suite fixes. The
upstream is pretty fast and responsive.

Are you able to sponsor the upload when we finish taking care of those
details? I've filled an RFS (#819289[7]), but forgot to add the
"debian-python" mailing list in "X-Debbugs-CC".

Regards,
Tiago.

[1]: https://github.com/myhro/deb-pythonpy/commit/f4ce711
[2]: https://github.com/myhro/deb-pythonpy/commit/868667b
[3]: https://github.com/myhro/deb-pythonpy/commit/3c2f4bd
[4]: https://github.com/Russell91/pythonpy/pull/79
[5]: https://lists.debian.org/debian-python/2016/03/msg00101.html
[6]: http://mentors.debian.net/package/pythonpy
[7]: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=3D819289

--=20
Tiago "Myhro" Ilieve
Blog: https://blog.myhro.info/
GitHub: https://github.com/myhro
LinkedIn: https://br.linkedin.com/in/myhro
Montes Claros - MG, Brasil