Bug#896704: RFS: python-picklable-itertools/0.1.1-2 [RC]

[Fabian Wolff]

On Fri, Apr 27, 2018 at 12:47:12PM -0400, Sergio Durigan Junior wrote:
> I've uploaded the package now.

Great, thank you!

Bug#896704: RFS: python-picklable-itertools/0.1.1-2 [RC]

[Fabian Wolff]

On Tue, Apr 24, 2018 at 10:55:00PM -0400, Sergio Durigan Junior wrote:
> Or you can also move the package under the Debian Python Modules Team
> umbrella, if it makes more sense.  Packaging Python modules with the
> DPMT is the preferred way nowadays, but that's really up to you (and
> just to be clear, I have no problem if you decide to maintain the
> package by yourself).

I have joined the DPMT and created a repository there:

  https://salsa.debian.org/python-team/modules/python-picklable-itertools

I have also set the DPMT as Maintainer in debian/control.

> I think it's best if we keep the package unchanged for now.  Sorry about
> this extra round-trip, but can you please re-add the Python 2 package?
> 
> Actually, I've just noticed that the lintian flag in question
> ("new-package-should-not-package-python2-module") only applies to the
> first upload of the package, and will actually be gone once we upload
> this next version, so there's really nothing that needs to be done from
> your part about it.  I'm sorry about the noise.
> 
> Once you reintroduce the Python 2 package, I'll do the upload.

I have reuploaded the package to Mentors:

  https://mentors.debian.net/package/python-picklable-itertools

Best regards,
Fabian

Bug#896704: RFS: python-picklable-itertools/0.1.1-2 [RC]

[Sergio Durigan Junior]

On Tuesday, April 24 2018, Fabian Wolff wrote:

> On Mon, Apr 23, 2018 at 08:58:30PM -0400, Sergio Durigan Junior wrote:
>> Hi Fabian,
>> 
>> I can help with it, but there are two things I'd like to see first.
>
> Thank you for your review!

My pleasure.  Thanks for the work on the package!

>> 1) There are no Vcs-* fields, and it's unclear to me where the git
>> repository for the package is located (I couldn't find it on
>> salsa.d.o).
>
> I did not maintain the package in a public Git repository so far, so I
> created a fresh repository, imported my changes and put it on Salsa:
>
>   https://salsa.debian.org/wolff-guest/python-picklable-itertools
>
> I have added Vcs-Git and Vcs-Browser fields accordingly.

Thanks, much appreciated.  I can also create a repository under the
Debian namespace on salsa if you want.

Or you can also move the package under the Debian Python Modules Team
umbrella, if it makes more sense.  Packaging Python modules with the
DPMT is the preferred way nowadays, but that's really up to you (and
just to be clear, I have no problem if you decide to maintain the
package by yourself).

>> 2) If having the Python 2 version of this package is important for some
>> reason, could you please override the lintian warning
>> ("new-package-should-not-package-python2-module")?
>
> I think that the Python 2 version is not really important; in fact, I
> did not even include it in the original package, but my previous
> sponsor for this package suggested that I should add it (see #841228).

Oh, well.  I don't want to go against Gianfranco here :-).  I understand
his reasoning, and actually there's a very recent thread developing on
debian-devel about this exact topic; see:

  

> I have now simply removed the Python 2 package from debian/control; is
> this sufficient, or do I have to do anything more than that?

I think it's best if we keep the package unchanged for now.  Sorry about
this extra round-trip, but can you please re-add the Python 2 package?

Actually, I've just noticed that the lintian flag in question
("new-package-should-not-package-python2-module") only applies to the
first upload of the package, and will actually be gone once we upload
this next version, so there's really nothing that needs to be done from
your part about it.  I'm sorry about the noise.

Once you reintroduce the Python 2 package, I'll do the upload.

Thanks,

-- 
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF  31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/


signature.asc
Description: PGP signature

Bug#896704: RFS: python-picklable-itertools/0.1.1-2 [RC]

[Fabian Wolff]

On Mon, Apr 23, 2018 at 08:58:30PM -0400, Sergio Durigan Junior wrote:
> Hi Fabian,
> 
> I can help with it, but there are two things I'd like to see first.

Thank you for your review!

> 1) There are no Vcs-* fields, and it's unclear to me where the git
> repository for the package is located (I couldn't find it on
> salsa.d.o).

I did not maintain the package in a public Git repository so far, so I
created a fresh repository, imported my changes and put it on Salsa:

  https://salsa.debian.org/wolff-guest/python-picklable-itertools

I have added Vcs-Git and Vcs-Browser fields accordingly.

> 2) If having the Python 2 version of this package is important for some
> reason, could you please override the lintian warning
> ("new-package-should-not-package-python2-module")?

I think that the Python 2 version is not really important; in fact, I
did not even include it in the original package, but my previous
sponsor for this package suggested that I should add it (see #841228).

I have now simply removed the Python 2 package from debian/control; is
this sufficient, or do I have to do anything more than that?


I have reuploaded the package to Mentors with the aforementioned
changes.

Best regards,
Fabian

Bug#896704: RFS: python-picklable-itertools/0.1.1-2 [RC]

[Sergio Durigan Junior]

Control: tags -1 + moreinfo
Control: owner -1 !

On Monday, April 23 2018, Fabian Wolff wrote:

> Package: sponsorship-requests
> Severity: important
>
> Dear mentors,
>
> I am looking for a sponsor for my package "python-picklable-itertools".
>
> The current version of this package is missing a dependency, which is
> why it fails to import on systems where that dependency is not
> installed (see #896213, #896214). dh-python did not find this
> dependency, and I apparently only tested the package on systems that
> happened to have the missing dependency installed, which is why
> importing worked fine for me.
>
> I have now manually added the missing dependency in debian/control.
> In order to (hopefully) avoid this problem in the future, I have also
> added a Testsuite field in debian/control.
>
> My changes are summarized in the latest changelog entry:
>
>   python-picklable-itertools (0.1.1-2) unstable; urgency=medium
>
> * Depend on python[3]-pkg-resources (Closes: #896213, #896214).
> * Add Testsuite field to debian/control.
> * Upgrade to Standards-Version 4.1.4 (no changes).
> * Upgrade to debhelper compat level 11 (no changes).
>
>-- Fabian Wolff   Mon, 23 Apr 2018 18:30:09 +0200

Hi Fabian,

I can help with it, but there are two things I'd like to see first.

1) There are no Vcs-* fields, and it's unclear to me where the git
repository for the package is located (I couldn't find it on
salsa.d.o).

2) If having the Python 2 version of this package is important for some
reason, could you please override the lintian warning
("new-package-should-not-package-python2-module")?

Thanks,

-- 
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF  31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/


signature.asc
Description: PGP signature

Bug#896704: RFS: python-picklable-itertools/0.1.1-2 [RC]

[Fabian Wolff]

Package: sponsorship-requests
Severity: important

Dear mentors,

I am looking for a sponsor for my package "python-picklable-itertools".

The current version of this package is missing a dependency, which is
why it fails to import on systems where that dependency is not
installed (see #896213, #896214). dh-python did not find this
dependency, and I apparently only tested the package on systems that
happened to have the missing dependency installed, which is why
importing worked fine for me.

I have now manually added the missing dependency in debian/control.
In order to (hopefully) avoid this problem in the future, I have also
added a Testsuite field in debian/control.

My changes are summarized in the latest changelog entry:

  python-picklable-itertools (0.1.1-2) unstable; urgency=medium

* Depend on python[3]-pkg-resources (Closes: #896213, #896214).
* Add Testsuite field to debian/control.
* Upgrade to Standards-Version 4.1.4 (no changes).
* Upgrade to debhelper compat level 11 (no changes).

   -- Fabian Wolff   Mon, 23 Apr 2018 18:30:09 +0200


The package is available on Mentors:

  https://mentors.debian.net/package/python-picklable-itertools
  
https://mentors.debian.net/debian/pool/main/p/python-picklable-itertools/python-picklable-itertools_0.1.1-2.dsc

Thank you!

Best regards,
Fabian