Your message dated Thu, 08 Feb 2018 18:06:39 +0000
with message-id <e1ejqal-0002a1...@fasolo.debian.org>
and subject line Bug#889592: fixed in lintian 2.5.74
has caused the Debian Bug report #889592,
regarding lintian: false positive for 
override_dh_auto_test-does-not-check-DEB_BUILD_PROFILES
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
889592: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=889592
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: lintian
Version: 2.5.73

On Sun, Feb 04, 2018 at 03:36:45PM +0530, Chris Lamb wrote:
> Some results coming in:
> 
>   
> https://lintian.debian.org/tags/override_dh_auto_test-does-not-check-DEB_BUILD_PROFILES.html
> 
> Thoughts?

Randomly checked few packages and they were mostly all fpos:

https://tracker.debian.org/media/packages/a/aff4/rules-0.24.post1-3
    Ok, this is crazy, he deserves what he got for being too clever with
    make :D

https://tracker.debian.org/media/packages/a/autoconf-dickey/rules-2.52%2B20170501-2
https://tracker.debian.org/media/packages/a/avro-c/rules-1.8.2-1
    Not sure why this is matched, I thought it wouldn't have been:
    |override_dh_auto_test:
    |   dh_auto_test --no-parallel

https://tracker.debian.org/media/packages/libx/libxcb/rules-1.12-1
    Similar to the previous:
    |override_dh_auto_test:
    |   dh_auto_test -- VERBOSE=1

https://tracker.debian.org/media/packages/o/osm2pgsql/rules-0.94.0%2Bds-1
    Also similar:
    |override_dh_auto_test:
    |   dh_auto_test || echo "Ignoring test failures"

https://tracker.debian.org/media/packages/a/akonadi-calendar-tools/rules-417.08.3-2
https://tracker.debian.org/media/packages/b/biosig4c%2B%2B/rules-1.4.1-1.1
    This seems to be a recurring pattern:
    |override_dh_auto_test:
    |   # Disable dh_auto_test at build time
    |   :

https://tracker.debian.org/media/packages/b/bibletime/rules-2.11.1-1
    I believe this is going the be another recurring pattern:
    |override_dh_auto_test:
    |   echo "Skip dh_auto_test because the tests rely on a display which is 
not there"


Whilst true that the last two technically should deal with
DEB_BUILD_OPTIONS=nocheck, I believe running a single `:` or a single
`echo` should be considered fine.


In my random pick, I saw this real positive:
https://tracker.debian.org/media/packages/l/lua-lgi/rules-0.9.2-1
|override_dh_auto_test:
|       make -C tests all
|       GI_TYPELIB_PATH=tests LD_LIBRARY_PATH=tests xvfb-run dh_auto_test
I recommend to add a similar thing to the testsuite, as this one is
*also* calling dh_auto_test, but it should still handle
DEB_BUILD_OPTIONS=nocheck by hand.

-- 
regards,
                        Mattia Rizzolo

GPG Key: 66AE 2B4A FCCF 3F52 DA18  4D18 4B04 3FCD B944 4540      .''`.
more about me:  https://mapreri.org                             : :'  :
Launchpad user: https://launchpad.net/~mapreri                  `. `'`
Debian QA page: https://qa.debian.org/developer.php?login=mattia  `-

Attachment: signature.asc
Description: PGP signature


--- End Message ---
--- Begin Message ---
Source: lintian
Source-Version: 2.5.74

We believe that the bug you reported is fixed in the latest version of
lintian, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 889...@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Chris Lamb <la...@debian.org> (supplier of updated lintian package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmas...@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Format: 1.8
Date: Thu, 08 Feb 2018 17:41:55 +0000
Source: lintian
Binary: lintian
Architecture: source all
Version: 2.5.74
Distribution: unstable
Urgency: medium
Maintainer: Debian Lintian Maintainers <lintian-ma...@debian.org>
Changed-By: Chris Lamb <la...@debian.org>
Description:
 lintian    - Debian package checker
Closes: 679124 738442 756130 762113 817170 858588 869547 889102 889486 889489 
889535 889591 889592 889628 889638 889639 889746 889760 889814 889856
Changes:
 lintian (2.5.74) unstable; urgency=medium
 .
   * Summary of tag changes:
     + Added:
       - control-tarball-compression-format
       - data-tarball-compression-format
       - debian-rules-is-dh_make-template
       - init.d-script-should-always-start-service
       - jar-contains-source
       - missing-systemd-service-for-init.d-script
       - override_dh_auto_test-does-not-check-DEB_BUILD_OPTIONS
       - source-contains-prebuilt-wasm-binary
       - spelling-error-in-patch-description
       - systemd-service-file-refers-to-unusual-wantedby-target
     + Renamed:
       - systemd-no-service-for-init-script ->
         omitted-systemd-service-for-init.d-script
       - systemd-no-service-for-init-rcS-script ->
         missing-systemd-service-for-init.d-rcS-script
     + Removed:
       - override_dh_auto_test-does-not-check-DEB_BUILD_PROFILES
 .
   * checks/changelog-file.desc:
     + [CL] Improve the long description of epoch-change-without-comment.
       Based on suggestions by Raphael Hertzog and Ian Jackson - thanks!
       (Closes: #889814)
   * checks/cruft.desc:
     + [BR] Check for wasm files.  (Closes: #889102)
     + [CL] Factor out call to _ships_examples to avoid excessive looping
       over $sorted_index.
     + [CL] Do not emit package-does-not-install-examples if we don't have
       any binary packages in our laboratory.  (Closes: #889591)
     + [CL] Improve the description of package-does-not-install-examples to
       give more debhelper advice.
     + [CL] Assume that if a source package generates a binary ending in
       "-examples" then it does ship examples.
   * checks/deb-format.{desc,pm}:
     + [CL] Add a classification tag for the .deb data tarball compression
       format.  (Closes: #738442)
     + [CL] Add a classification tag for the control tarball compression
       format.  (Closes: #889856)
   * checks/fields.pm:
     + [CL] Avoid false positives when checking binary packages depending on
       toolchain packages by ignoring packages starting with "dh-" or ending
       with "-source". Thanks to Josh Triplett for the report.
       (Closes: #889486)
   * checks/files.pm:
     + [BR] Add context for privacy breach in order to improve debugging.
   * checks/fields.desc:
     + [CL] Downgrade severity of build-depends-on-obsolete-package from
       error to warning.  (Closes: #889638)
   * checks/java.{desc,pm}:
     + [CL] Only warn about bad-jar-name for "public" .jar files.
       (Closes: #889628)
     + [CL] Check for .jar files that embed Foo.java alongside a Foo.class
       file.  (Closes: #762113)
   * checks/init.d.{desc,pm}:
     + [CL] Warn about packages that use ENABLED="true" (etc.) in
       /etc/default files.
   * checks/patch-systems.{desc,pm}:
     + [CL] Avoid emitting "Can't use an undefined value as an ARRAY
       reference" warnings when debian/patches is a file, not a directory.
       (Closes: #889535)
     + [CL] Check spelling of patch headers.  (Closes: #756130)
   * checks/rules.{desc,pm}:
     + [CL] Fix a number of false-positives when checking the
       "override_dh_auto_test-does-not-check-DEB_BUILD_PROFILES" tag
       (Closes: #889592)
     + [CL] Make a large number of changes suggested by Mattia Rizzolo to
       the override_dh_auto_test-does-not-check-DEB_BUILD_PROFILES tag,
       renaming it to reference DEB_BUILD_OPTIONS throughout, add Debian
       Policy 4.9.1 to the tag's Ref, lower "Certanty" to "wild-guess" and
       mark the tag as experimental, updating the tests to match.
       (Closes: #889746)
     + [CL] Check for debian/rules files that are dh_make templates.
       (Closes: #679124)
   * checks/scripts.desc:
     + [CL] Improve, elaborate and tidy the long description of the
       maintainer-script-should-not-use-recursive-chown-or-chmod tag.
       Heavily based on a patch by Daniel Kahn Gillmor - thanks!
       (Closes: #889489)
   * checks/source-copyright.pm:
     + [CL] Prevent false positives when checking for missing NOTICE.txt
       files by looking inside .jar archives.  (Closes: #889760)
   * checks/systemd.{desc,pm}:
     + [CL] Warn about unit files that install to usual WantedBy= targets.
       Thanks to Sam Morris for the initial patch.  (Closes: #817170)
     + [CL] Rework the no service detection, improving the (rarely
       overridden) tag names to better match what they detect as well as
       adding a new "missing-systemd-service-for-init.d-script" pedantic tag
       where we do not have an equivalent unit as this implies missing
       bespoke security hardening support, etc. Thanks to Lucas Nussbaum for
       his input.  (Closes: #858588)
   * checks/udev.pm:
     + [CL] Add simple GOTO parsing to avoid false positives when checking
       for udev rules for SUBSYSTEM specifiers.  (Closes: #869547, #889639)
 .
   * commands/reporting-{html-reports,lintian-harness}.pm:
     + [NT] Register packages that fail during archive wide processing.
 .
   * data/files/privacy-breaker-fragments:
     + [BR] Detect new fragments for Google CSE.
   * data/spelling/corrections:
     + [PW] Add a number of corrections.
 .
   * lib/Lintian/Util.pm:
     + [NT] Give lower processing priority to packages that repeatedly
       trigger errors during archive-wide processing.
 .
   * reporting/templates/index.tmpl:
     + [NT] Display summary of how many groups had errors during their
       last processing.
   * reporting/templates/{lintian.css,maintainer}.tmpl:
     + [NT] Use a distinct error status instead of "Outdated" for
       packages with errors during their last processing.
Checksums-Sha1:
 0acdfd1029057b3db40203d97e6b4271fee20e95 3505 lintian_2.5.74.dsc
 3b6d9daa7eb6ce8ae2c6e35acf94d6e1f0be397d 1479936 lintian_2.5.74.tar.xz
 cf0a6e3875994c42d5721025470cbc869f2bbcae 1100980 lintian_2.5.74_all.deb
 a6ee207b9b6dcfa92a7d1edf109627e97ab2006e 15989 lintian_2.5.74_amd64.buildinfo
Checksums-Sha256:
 e13f7a748cc513435119e0eb20e50a6612600c6ce57179262ce90b25f440323c 3505 
lintian_2.5.74.dsc
 7bc82aeddd3092b48a357a7c877d9dc5d49b8c16af8b50f25de80b3f6f9ef847 1479936 
lintian_2.5.74.tar.xz
 fe39422089d32f2e6bb010c7ff03f013ab56f2bc0b7a398972625d74c405ddf7 1100980 
lintian_2.5.74_all.deb
 f7967fbf4ca5ee4c901be34aa0b2f940704bc46bdae56fb7f0df698a9f7011b5 15989 
lintian_2.5.74_amd64.buildinfo
Files:
 a93cb16d31e53f38073ab6af042d5299 3505 devel optional lintian_2.5.74.dsc
 34e57c8fbbda0693bf901a4fe7f26903 1479936 devel optional lintian_2.5.74.tar.xz
 77baff5d246ff4f9ac449146c88116ff 1100980 devel optional lintian_2.5.74_all.deb
 3c9622e2cbdc75c505cef7afa3793cd0 15989 devel optional 
lintian_2.5.74_amd64.buildinfo

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEEwv5L0nHBObhsUz5GHpU+J9QxHlgFAlp8kDAACgkQHpU+J9Qx
Hliavg/5AV0Rc8AC70iDmi51z4lo6lweqk0usfwiEc7nw6eQUJYsguKs4sZFmiRM
1BrtvcG+XPOXt5BtjdPvpjRW4C9rIk63JMJT5iO/RzWxu94z3iDuie4RDZxFWAoB
gtMlBRMz475/LWsaXBFGzLW3z/yj1qLnXUuza/X7fy8sIRdun94RKObMXvfRksd3
dcxpaqpiKZ966DvZTYsR7Lc5ajOMS7gDpxSO5c1WY9VJGmThgpaWI5+dAVSfoVcj
I9tnqmX1HfuqVGCVFutKKiRbZ6Uoxg2y/cITlqYotD0xx/UnrjJ/O0E2jrV5a+WW
xVD/DqW53Q+wM8T7uaglFlGYbvnJHz/J/L2fSstcCmJcBii6fbXgsSdts0FPUKrf
xWvHokw3Ndz9mlNGTOlnwFOCCyXEQms+vBsKlkX/E088Bc5UZdaeszaWYpu5Ym/P
mT6TDgun1K6uKHiBxhZv+hWf0omyyMYMl6+FAX/NE/R6fQKJ479VUd+keXYsfn3y
/dbfV4pNI0GLEwhWEL7b85GWzd2BHFiDlekax9+wLXZgM97BRn/ievldNf5hbp5W
Wv9r/2yl1eF9x03ao6JG/9YbBEeLvzjYdEFibcgp5f6IJt7kBU07mgCX/dwaCrbL
T7gbeW+xdnnRQz/Z8968bU2fOynAOumR6/rXZutp5839OXJoVPM=
=ocH3
-----END PGP SIGNATURE-----

--- End Message ---

Reply via email to