Bug#1054571: devscripts: debootsnap: Missing Recommends on python3-pycurl

2023-11-30 Thread Michael Prokop
* David (Plasma) Paul [Wed Nov 29, 2023 at 03:47:50PM -0600]:
> On Tue, 28 Nov 2023 12:48:31 +0100 Michael Prokop  wrote:
> > * David (Plasma) Paul [Wed Oct 25, 2023 at 11:36:48PM -0500]:

[...]

> > And the package description for says:
> > 
> > | - debootsnap: Combines debootstrap and snapshot.debian.org to create a 
> > chroot
> > |   containing exactly the requested selection of packages. This can be used
> > |   to re-create a chroot from the past, for example to reproduce a bug. The
> > |   tool is also used by debrebuild to build a package in a chroot with 
> > build
> > |   dependencies in the same version as recorded in the buildinfo file.
> > |   [apt-utils, dpkg-dev, equivs, mmdebstrap, python3-pycurl]
> > 
> > I can confirm that apt-utils, equivs, mmdebstrap and python3-pycurl,
> > but also python3-debian and python3-requests seem to be required.
> > 
> > Given that e.g. `equivs` is listed in both Recommends, but also in
> > the `[...]` section of the debootsnap's description, maybe the list
> > should be updated to:
> > 
> >   [apt-utils equivs mmdebstrap python3-debian python3-pycurl 
> > python3-requests]
> > 
> > ?
> 
> I concur. Given that dpkg-dev is already listed as a hard Dependency of
> devscripts and given that debootsnap indeed requires python3-debian and
> python3-requests, the list of packages following 'debootsnap' in the
> Description stanza of devscripts' metadata should read:
> 
> [apt-utils, equivs, mmdebstrap, python3-debian, python3-pycurl, 
> python3-requests]

*nod*

I implemented this at
https://salsa.debian.org/debian/devscripts/-/merge_requests/374

regards
-mika-


signature.asc
Description: PGP signature


Bug#1054571: devscripts: debootsnap: Missing Recommends on python3-pycurl

2023-11-29 Thread David (Plasma) Paul
On Tue, 28 Nov 2023 12:48:31 +0100
Michael Prokop  wrote:

> Hi,
> 
> * David (Plasma) Paul [Wed Oct 25, 2023 at 11:36:48PM -0500]:
> > Package: devscripts
> > Version: 2.21.6
> > Severity: normal
> > 
> > Dear Maintainer,
> > 
> > The devscripts binary package is missing a Recommends on
> > python3-pycurl which is a dependency of the debootsnap script.  
> 
> So in devscripts v2.23.6 we have:

> | Depends: dpkg-dev (>= 1.18.19), fakeroot, file, gnupg | gnupg2,
> gpgv | gpgv2, libfile-dirlist-perl, libfile-homedir-perl,
> libfile-touch-perl, libfile-which-perl, libipc-run-perl, libmoo-perl,
> libwww-perl, patchutils, sensible-utils, wdiff, perl:any, python3:any

> | Recommends: apt (>= 1.3~pre3), dctrl-tools, dput | dupload,
> libdistro-info-perl, libdpkg-perl, libencode-locale-perl,
> libgit-wrapper-perl, libgitlab-api-v4-perl, libjson-perl,
> liblist-compare-perl, libstring-shellquote-perl, libtry-tiny-perl,
> liburi-perl, licensecheck, lintian, man-db, patch, python3-apt,
> python3-debian (>= 0.1.15), python3-magic, python3-requests,
> python3-unidiff, python3-xdg, strace, unzip, wget | curl, xz-utils,
> debian-keyring, equivs, liblwp-protocol-https-perl,
> libsoap-lite-perl, pristine-tar

> | Suggests: adequate, at, autopkgtest, bls-standalone, bsd-mailx |
> mailx, build-essential, check-all-the-things, cvs-buildpackage,
> debhelper, diffoscope, disorderfs, dose-extra (>= 4.0), duck,
> elpa-devscripts, faketime, gnuplot, how-can-i-help,
> libauthen-sasl-perl, libdbd-pg-perl, libfile-desktopentry-perl,
> libterm-size-perl, libtimedate-perl, libyaml-syck-perl, mmdebstrap,
> mozilla-devscripts, mutt, piuparts, postgresql-client, pristine-lfs,
> quilt, ratt, reprotest, ssh-client, svn-buildpackage, w3m

> And the package description for says:
> 
> | - debootsnap: Combines debootstrap and snapshot.debian.org to create a 
> chroot
> |   containing exactly the requested selection of packages. This can be used
> |   to re-create a chroot from the past, for example to reproduce a bug. The
> |   tool is also used by debrebuild to build a package in a chroot with build
> |   dependencies in the same version as recorded in the buildinfo file.
> |   [apt-utils, dpkg-dev, equivs, mmdebstrap, python3-pycurl]
> 
> I can confirm that apt-utils, equivs, mmdebstrap and python3-pycurl,
> but also python3-debian and python3-requests seem to be required.
> 
> Given that e.g. `equivs` is listed in both Recommends, but also in
> the `[...]` section of the debootsnap's description, maybe the list
> should be updated to:
> 
>   [apt-utils equivs mmdebstrap python3-debian python3-pycurl python3-requests]
> 
> ?

I concur. Given that dpkg-dev is already listed as a hard Dependency of
devscripts and given that debootsnap indeed requires python3-debian and
python3-requests, the list of packages following 'debootsnap' in the
Description stanza of devscripts' metadata should read:

[apt-utils, equivs, mmdebstrap, python3-debian, python3-pycurl, 
python3-requests]

-- 
Plasma



Bug#1054571: devscripts: debootsnap: Missing Recommends on python3-pycurl

2023-11-28 Thread Michael Prokop
Hi,

* David (Plasma) Paul [Wed Oct 25, 2023 at 11:36:48PM -0500]:
> Package: devscripts
> Version: 2.21.6
> Severity: normal
> 
> Dear Maintainer,
> 
> The devscripts binary package is missing a Recommends on python3-pycurl
> which is a dependency of the debootsnap script.

So in devscripts v2.23.6 we have:

| Depends: dpkg-dev (>= 1.18.19), fakeroot, file, gnupg | gnupg2, gpgv | gpgv2, 
libfile-dirlist-perl, libfile-homedir-perl, libfile-touch-perl, 
libfile-which-perl, libipc-run-perl, libmoo-perl, libwww-perl, patchutils, 
sensible-utils, wdiff, perl:any, python3:any
| Recommends: apt (>= 1.3~pre3), dctrl-tools, dput | dupload, 
libdistro-info-perl, libdpkg-perl, libencode-locale-perl, libgit-wrapper-perl, 
libgitlab-api-v4-perl, libjson-perl, liblist-compare-perl, 
libstring-shellquote-perl, libtry-tiny-perl, liburi-perl, licensecheck, 
lintian, man-db, patch, python3-apt, python3-debian (>= 0.1.15), python3-magic, 
python3-requests, python3-unidiff, python3-xdg, strace, unzip, wget | curl, 
xz-utils, debian-keyring, equivs, liblwp-protocol-https-perl, 
libsoap-lite-perl, pristine-tar
| Suggests: adequate, at, autopkgtest, bls-standalone, bsd-mailx | mailx, 
build-essential, check-all-the-things, cvs-buildpackage, debhelper, diffoscope, 
disorderfs, dose-extra (>= 4.0), duck, elpa-devscripts, faketime, gnuplot, 
how-can-i-help, libauthen-sasl-perl, libdbd-pg-perl, libfile-desktopentry-perl, 
libterm-size-perl, libtimedate-perl, libyaml-syck-perl, mmdebstrap, 
mozilla-devscripts, mutt, piuparts, postgresql-client, pristine-lfs, quilt, 
ratt, reprotest, ssh-client, svn-buildpackage, w3m

And the package description for says:

| - debootsnap: Combines debootstrap and snapshot.debian.org to create a chroot
|   containing exactly the requested selection of packages. This can be used
|   to re-create a chroot from the past, for example to reproduce a bug. The
|   tool is also used by debrebuild to build a package in a chroot with build
|   dependencies in the same version as recorded in the buildinfo file.
|   [apt-utils, dpkg-dev, equivs, mmdebstrap, python3-pycurl]

I can confirm that apt-utils, equivs, mmdebstrap and python3-pycurl,
but also python3-debian and python3-requests seem to be required.

Given that e.g. `equivs` is listed in both Recommends, but also in
the `[...]` section of the debootsnap's description, maybe the list
should be updated to:

  [apt-utils equivs mmdebstrap python3-debian python3-pycurl python3-requests]

?

regards
-mika-


signature.asc
Description: PGP signature