Re: pdm-pep517: Shall we package it now?

[Julian Gilbey <julian@d-and-j.net>]

On Wed, Jun 29, 2022 at 10:58:48AM -0400, Boyuan Yang wrote:
> Hi,
> [...]
> Thanks. I have uploaded an initial version to the NEW queue. The packaging
> work is at https://salsa.debian.org/python-team/packages/pdm-pep517/ .
> 
> It is worth noting that I took the very aggressive way in stripping every
> vendored libraries, see
> https://salsa.debian.org/python-team/packages/pdm-pep517/-/tree/master/debian/patches
> . This may not be ideal since the patch will need human intervention for every
> new upstream release, but anyway let's have a working version in Sid first.
> Meanwhile, any review or other types of help would be appreciated.

Hi Boyuan,

I've just taken a look; the exclusions and patching look fine to me.
Yes, there will need to be some human intervention, but there always
should be for new upstream versions.  It may well be that all that is
needed is to refresh the patch and run:

quilt push -a
rgrep _vendor pdm

(or the equivalent gbp pq push or something like that) to check that
nothing has changed.

One small change to the patch:

* In pdm/pep517/utils.py and pdm/pep517/wheel.py, there is no need to
  insert an "import packaging"

Best wishes,

   Julian