Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.debian.maint.python > #14559

Re: review for pipenv/2022.10.12-1

From Jeroen Ploemen <jcfp@debian.org>
Newsgroups linux.debian.maint.python
Subject Re: review for pipenv/2022.10.12-1
Date 2022-10-27 19:20 +0200
Message-ID <FleHT-h7II-1@gated-at.bofh.it> (permalink)
References <FjqDv-fZP8-1@gated-at.bofh.it> <FkPTb-gRY4-5@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


[Multipart message — attachments visible in raw view] - view raw

On Wed, 26 Oct 2022 17:29:59 +0300
Ileana Dumitrescu <ileanadumitrescu95@gmail.com> wrote:

> Thank you for the feedback! I made changes as you suggested. There
> is a new upstream version that I also included in the new package.

Great! The copyright stuff is a chore on packages like this, so
thanks alot for seeing that through.

> Reading debian package policy I noticed that removing files from a
> tarball for a repack (as Bastian suggested in bug #1019714) should
> require a +ds suffix, so I packaged the new version with
> 2022.10.25+ds-1. Please let me know if I did this incorrectly or if
> this should not be done for this package.

Indeed, a repacksuffix is used to indicate changes were made to an
upstream release so that's perfectly fine this way. Typically, +dfsg
is used to signal the source was repacked for DFSG compliance reasons
and +ds when repacking for some other reason.

I did just notice the upstream release contains several other files
worth considering for removal: a bunch of windows executables [1].
 
> > + E: pipenv: python-traceback-in-manpage is a false positive,  
>     please override.
> 
> This did not show up in lintian with the new upstream version.

It seems they revamped the manpage, although the new one also earns a
lintian hit [1], this time about a bad (missing?) 'whatis' entry.

Lintian seems to think the source for some html file is missing, but
at first glance that hit may well be a false positive triggered by
some bits of javascript.


Unrelated to any of the above, I pushed some minor changes and
enabled the CI on salsa.


[1]https://salsa.debian.org/python-team/packages/pipenv/-/jobs/3434663

Back to linux.debian.maint.python | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

review for pipenv/2022.10.12-1 Jeroen Ploemen <jcfp@debian.org> - 2022-10-22 19:40 +0200
  Re: review for pipenv/2022.10.12-1 Ileana Dumitrescu <ileanadumitrescu95@gmail.com> - 2022-10-26 16:50 +0200
    Re: review for pipenv/2022.10.12-1 Jeroen Ploemen <jcfp@debian.org> - 2022-10-27 19:20 +0200
      Re: review for pipenv/2022.10.12-1 Ileana Dumitrescu <ileanadumitrescu95@gmail.com> - 2022-10-28 16:10 +0200
        Re: review for pipenv/2022.10.12-1 Jeroen Ploemen <jcfp@debian.org> - 2022-10-29 09:30 +0200
          Re: review for pipenv/2022.10.12-1 Ileana Dumitrescu <ileanadumitrescu95@gmail.com> - 2022-10-31 15:50 +0100
  Re: review for pipenv/2022.10.12-1 Bastian Venthur <venthur@debian.org> - 2022-10-28 18:20 +0200

csiph-web