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

Martin Pitt mpitt at debian.org
Mon Jul 27 09:56:28 BST 2015


Hey Niels,

sorry for the delay, I was on holiday.

Niels Thykier [2015-07-15  7:20 +0200]:
> 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
> """

Thanks for pointing out! Fixed in git now:

  http://anonscm.debian.org/cgit/autopkgtest/autopkgtest.git/commit/?id=5521c

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

Using whitespace only is still legitimate syntax and I don't want to
break that for the time being due to backwards compat. Likewise I
wouldn't want to introduce potentially hundreds of lintian warnings
just for that yet. So a lintian warning seems to be too strong to me,
at least for the time being.

Thanks!

Martin

-- 
Martin Pitt                        | http://www.piware.de
Ubuntu Developer (www.ubuntu.com)  | Debian Developer  (www.debian.org)




More information about the Pkg-systemd-maintainers mailing list