[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#798359: lintian: list of autopkgtest restrictions is not up-to-date



Control: retitle -1 lintian: unknown-runtime-tests-restriction reports comma-delimited known restrictions

Hi again,

Axel Beckert wrote:
> Jérémy Bobbio wrote:
> > It seems that the list of known restrictions for debian/tests/control is
> > not up-to-date.
> > 
> > Several packages raise unknown-runtime-tests-restriction [1] despite
> > using restrictions properly listed in the reference documentation [2].
> > 
> >  [1]: https://lintian.debian.org/tags/unknown-runtime-tests-restriction.html
> >  [2]: https://anonscm.debian.org/gitweb/?p=autopkgtest/autopkgtest.git;a=blob_plain;f=doc/README.package-tests.rst;hb=HEAD
> 
> Hrm. I just compared the source code with [2]. The source code is
> uptodate. There is probably a bug somewhere else in the logic.

I've taken the following example:

→ cat -n cedar-backup2-2.26.4/debian/tests/control
     1  # Run the unit tests on the library code
     2  Tests: unittest
     3  Restrictions: allow-stderr
     4
     5  # Run some basic tests on command-line functionality for cback
     6  Tests: cback-version cback-help cback-diagnostics
     7  Restrictions: allow-stderr
     8
     9  # Run some basic tests on command-line functionality for cback-span
    10  Tests: cback-span-version cback-span-help cback-span-diagnostics
    11  Restrictions: allow-stderr
    12
    13  # Run some basic tests on command-line functionality for cback-amazons3-sync
    14  Tests: cback-amazons3-sync-version cback-amazons3-sync-help cback-amazons3-sync-diagnostics
    15  Restrictions: allow-stderr
    16
    17  # Run a simple backup and make sure it succeeds
    18  Tests: simplebackup
    19  Depends: @, awk, diffutils, tar
    20  Restrictions: allow-stderr, needs-root
    21

Lintian reports:

P: cedar-backup2 source: unknown-runtime-tests-restriction allow-stderr, paragraph starting at line 18

But allow-stderr is in all Restrictions, not only in the one in line
18. So it seems the comma after "allow-stderr" which seems to confuse
lintian (as it also emits the comma in the warning).

Will try to find a fix.

Thanks for the report!

		Regards, Axel
-- 
 ,''`.  |  Axel Beckert <abe@debian.org>, http://people.debian.org/~abe/
: :' :  |  Debian Developer, ftp.ch.debian.org Admin
`. `'   |  4096R: 2517 B724 C5F6 CA99 5329  6E61 2FF9 CD59 6126 16B5
  `-    |  1024D: F067 EA27 26B9 C3FC 1486  202E C09E 1D89 9593 0EDE


Reply to: