Bug#792418: Wrong "illegal-runtime-test-name" for comma separated Tests: in debian/tests/control

[Niels Thykier]

On 2015-07-14 18:01, Martin Pitt wrote:
> Package: lintian
> Version: 2.5.33
> 
> Hello,
> 
> lintian currently complains about the systemd source package:
> 
>   W: systemd source: illegal-runtime-test-name timedated, paragraph starting at line 1
> 
> and others, see
> https://lintian.debian.org/maintainer/pkg-systemd-maintainers@lists.alioth.debian.org.html#systemd
> 
> This is because the file looks like this:
> 
>   Tests: timedated, hostnamed, localed-locale, localed-x11-keymap
>   Depends: systemd, libpam-systemd, acl, locales
>   Restrictions: needs-root, isolation-container
> 
> Indeed the spec [1] said "whitespace separated" until today, but that
> was obsolete -- I added comma separation a fair while ago to be
> consistent with debian/control (Depends: and everything else is also
> comma separated). So I fixed that in [2] and updated [1].
> 
> Thanks!
> 
> Martin
> 
> [1] https://people.debian.org/~mpitt/autopkgtest/README.package-tests.html
> [2] http://anonscm.debian.org/cgit/autopkgtest/autopkgtest.git/commit/?id=a0595891
> 

Hi Martin,

Thanks for reporting the bug.

If you are planning on using "comma" as the canonical separator, please
consider also updating the examples in your documentation.  E.g. your
[1] still says:

"""
The source package provides a test metadata file debian/tests/control.
This is a file containing zero or more RFC822-style stanzas, along these
lines:

Tests: fred bill bongo
Restrictions: needs-root breaks-testbed
"""

On a related note, do you want us to recommend users to use comma
instead of whitespace (by warning about a pure whitespace separator)?

Thanks,
~Niels