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

Axel Beckert abe at debian.org
Tue Sep 8 20:25:50 BST 2015


Control: tag -1 + pending

Hi Martin

Martin Pitt wrote:
> 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

Fixed in lintian's git now, too:

https://anonscm.debian.org/cgit/lintian/lintian.git/commit/?id=98c2644e

> > 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.

Please file a separate bug report against lintian once
whitespace-only as delimiter gets deprecated.

		Regards, Axel
-- 
 ,''`.  |  Axel Beckert <abe at 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



More information about the Pkg-systemd-maintainers mailing list