[Debian-med-packaging] Bug#904776: /usr/bin/dpkg-source: please parse d/t/control.autodep8 like d/t/control

Paul Gevers elbrus at debian.org
Fri Jul 27 21:04:24 BST 2018


Package: dpkg-dev
Version: 1.19.0.5
Severity: normal
File: /usr/bin/dpkg-source
X-Debbugs-CC: debian-ci at lists.debian.org, python-cobra at packages.debian.org
User: debian-ci at lists.debian.org
Usertags: issue
Control: affects -1 src:python-cobra

Dear dpkg maintainers,

In commit 90324cf support was added to dpkg-source to read debian/tests/control
files and convert all test dependencies into a Testsuite-Triggers field. This
works great and is used by britney to do unstable-to-testing migration
tests. However, we recently spotted a regression (in python-cobra) that was
missed, because it uses autodep8 with additional test cases. This is done in
line with the autodep8(1) man page¹:
'''
COMBINING AUTO-GENERATED TESTS WITH MANUALLY SPECIFIED ONES
       If `debian/tests/control.autodep8` exists, autodep8 will prepend the
       contents of that file to its own output. In that case, autodep8 will
       exit with a status of 0 even if no known package type is detected.
'''

It would be great if dpkg-source would also parse debian/tests/control.autodep8
in the same way as it does debian/tests/control, except I guess it should not
set the Testsuite field (as dpkg-source doesn't know which type of autopkgtest
this is).

I cloned the dpkg source, but as it is written in Perl, I felt uncomfortable
providing a patch. I may reconsider if this is found valuable nevertheless.

Thanks for all your good work.

Paul
¹ https://manpages.debian.org/stretch/autodep8/autodep8.1.en.html

-- System Information:
Debian Release: buster/sid
  APT prefers testing-debug
  APT policy: (500, 'testing-debug'), (500, 'testing'), (200, 'testing'), (50, 'testing')
Architecture: amd64 (x86_64)

Kernel: Linux 4.17.0-1-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), LANGUAGE=en_US:en (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages dpkg-dev depends on:
ii  binutils      2.31.1-1
ii  bzip2         1.0.6-8.1
ii  libdpkg-perl  1.19.0.5
ii  make          4.2.1-1.1
ii  patch         2.7.6-2
ii  perl          5.26.2-6
ii  tar           1.30+dfsg-2
ii  xz-utils      5.2.2-1.3

Versions of packages dpkg-dev recommends:
ii  build-essential          12.5
ii  fakeroot                 1.23-1
ii  gcc                      4:8.1.0-1
ii  gcc-5 [c-compiler]       5.4.1-4
ii  gcc-6 [c-compiler]       6.4.0-18
ii  gcc-7 [c-compiler]       7.3.0-27
ii  gcc-8 [c-compiler]       8.1.0-12
ii  gnupg                    2.2.9-1
ii  gnupg2                   2.2.9-1
ii  gpgv                     2.2.9-1
ii  libalgorithm-merge-perl  0.08-3

Versions of packages dpkg-dev suggests:
ii  debian-keyring  2018.06.24

-- no debconf information

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://alioth-lists.debian.net/pipermail/debian-med-packaging/attachments/20180727/947a3d4e/attachment.sig>


More information about the Debian-med-packaging mailing list