Bug#982871: lintian-brush: Should not set "Testsuite: autopkgtest-pkg-perl" if debian/tests/control is present

[Axel Beckert]

Hi Jelmer,

Jelmer Vernooij wrote:
> > So please stop adding "Testsuite:" headers if debian/tests/control is
> > already present.
> > 
> > Luckily the testsuite declared in debian/tests/control was still run, so
> > it didn't completely break autopkgtest for this package, but at least
> > caused unnecessary tests being tried to run, but then skipped.
> Thanks for the bug report.
> 
> It looks like this is also a bug in lintian, since it reports
> team/pkg-perl/testsuite/no-team-tests for equivs:
> 
> https://lintian.debian.org/tags/team/pkg-perl/testsuite/no-team-tests.html

Hmmm, I have recently worked on debsums (similar case of a native
pkg-perl-team package which also has a debian/tests/control file) and
I can't remember having seen that warning. Then again I had the
feeling that the lintian tags on the web are often outdated.

Just checked: Yeah, it's overridden there. (Overridden by myself in
2015. :-)

But the lintian website is outdated nevertheless. It doesn't list
debsums 3.0.1 from over 2 months ago and still lists "E
malformed-override Unknown tag nonteam-testsuite-header in line 3"
which was the old name of that tag — fixed in the 3.0.1 upload.

And yeah, it emits that for equivs as well:

W: equivs source: team/pkg-perl/testsuite/no-team-tests autopkgtest

I should probably override that also for equivs. Does lintian-brush
respect lintian overrides in the meanwhile?

Will file a bug report against lintian, too. Thanks for these hints!

		Regards, Axel
-- 
 ,''`.  |  Axel Beckert <abe at debian.org>, https://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