Bug#692282: [new check] debian/tests/control but not (XS-)Testsuite: autopkgtest header in debian/control

2013-02-12 Thread Niels Thykier
On 2013-02-08 18:13, Nicolas Boulenguez wrote: I wrote at some point: Thanks for the code. I have attached a small diff for some stylistic things (note the test .desc changes also prevents a synopsis not phrased properly tag). For the record, the my-changes.diff was intended to be an

Bug#692282: [new check] debian/tests/control but not (XS-)Testsuite: autopkgtest header in debian/control

2013-02-08 Thread Niels Thykier
On 2013-01-25 20:46, Nicolas Boulenguez wrote: Package: lintian Followup-For: Bug #692282 Checks for syntax errors/unparseable control file. Check that all paragraphs have a Tests field. Or even check that all the restrictions are known. Here is a suggestion for review. Hi, Thanks for

Bug#692282: [new check] debian/tests/control but not (XS-)Testsuite: autopkgtest header in debian/control

2013-02-08 Thread Nicolas Boulenguez
Please see test.log. Sorry, I did not detect these problems because I test with 2.5.11 and --root option. Three pedantic messages were not reported, I copied similar options from another test to t/tests/testsuite-general/desc to unmask them. Path debian/tests/fifo is reported missing instead

Bug#692282: [new check] debian/tests/control but not (XS-)Testsuite: autopkgtest header in debian/control

2013-01-25 Thread Nicolas Boulenguez
Package: lintian Followup-For: Bug #692282 Checks for syntax errors/unparseable control file. Check that all paragraphs have a Tests field. Or even check that all the restrictions are known. Here is a suggestion for review. From 0e6b9a16707dab1d32ee772da0b3ed81e2907a49 Mon Sep 17 00:00:00

Bug#692282: [new check] debian/tests/control but not (XS-)Testsuite: autopkgtest header in debian/control

2013-01-22 Thread Niels Thykier
Control: tags -1 pending On 2013-01-21 00:46, Nicolas Boulenguez wrote: Package: lintian Followup-For: Bug #692282 if (not ($info-index ('debian/tests')-is_dir and $control-is_regular_file)) { directories must have a trailing / (i.e. 'debian/tests/'). Not sure to understand what you

Bug#692282: [new check] debian/tests/control but not (XS-)Testsuite: autopkgtest header in debian/control

2013-01-20 Thread Nicolas Boulenguez
Package: lintian Version: 2.5.11 Followup-For: Bug #692282 Happy new year, and thanks for your advices. Here is a new attempt. Adding the Testsuite field in data/common/source-fields causes the generated profiles/debian/main.profile to change during build. It should either be updated in the

Bug#692282: [new check] debian/tests/control but not (XS-)Testsuite: autopkgtest header in debian/control

2013-01-20 Thread Niels Thykier
On 2013-01-20 11:39, Nicolas Boulenguez wrote: Package: lintian Version: 2.5.11 Followup-For: Bug #692282 Happy new year, and thanks for your advices. Here is a new attempt. Hi, Thanks for looking into this. Adding the Testsuite field in data/common/source-fields causes the generated

Bug#692282: [new check] debian/tests/control but not (XS-)Testsuite: autopkgtest header in debian/control

2013-01-20 Thread Nicolas Boulenguez
Package: lintian Followup-For: Bug #692282 if (not ($info-index ('debian/tests')-is_dir and $control-is_regular_file)) { directories must have a trailing / (i.e. 'debian/tests/'). Not sure to understand what you mean by must. On my machine, $info-index ('debian/tests')-is_dir returns true

Bug#692282: [new check] debian/tests/control but not (XS-)Testsuite: autopkgtest header in debian/control

2013-01-07 Thread Christoph Berg
Re: Stefano Zacchiroli 2012-11-04 20121104170013.30086.35818.reportbug@usha.takhisis.invalid Can you please add a lintian test that will warn if a debian/tests/control file exists, but no XS-Testsuite: autopkgtest header is present in the source stanza of debian/control ? And of course, the

Bug#692282: [new check] debian/tests/control but not (XS-)Testsuite: autopkgtest header in debian/control

2013-01-07 Thread Christoph Berg
Re: To Stefano Zacchiroli 2013-01-07 20130107155757.ga28...@msgid.df7cb.de Re: Stefano Zacchiroli 2012-11-04 20121104170013.30086.35818.reportbug@usha.takhisis.invalid Can you please add a lintian test that will warn if a debian/tests/control file exists, but no XS-Testsuite: autopkgtest

Bug#692282: [new check] debian/tests/control but not (XS-)Testsuite: autopkgtest header in debian/control

2013-01-07 Thread Niels Thykier
On 2012-11-23 23:42, Nicolas Boulenguez wrote: Package: lintian Version: 2.5.10.2 Tags: patch Followup-For: Bug #692282 Please consider the attached patch. Hi, Thanks for looking at writing at a patch and sorry for the delay in getting back to you. I hope I can convince you to do a

Bug#692282: [new check] debian/tests/control but not (XS-)Testsuite: autopkgtest header in debian/control

2012-11-23 Thread Nicolas Boulenguez
Package: lintian Version: 2.5.10.2 Tags: patch Followup-For: Bug #692282 Please consider the attached patch. diff -rNu ../old/lintian-2.5.10.2/checks/control-file ./checks/control-file --- ../old/lintian-2.5.10.2/checks/control-file 2012-09-17 11:56:05.0 +0200 +++ ./checks/control-file

Bug#692282: [new check] debian/tests/control but not (XS-)Testsuite: autopkgtest header in debian/control

2012-11-04 Thread Stefano Zacchiroli
Package: lintian Version: 2.5.10.2 Severity: wishlist [ context is https://lists.debian.org/debian-qa/2012/11/msg9.html where we are discussing the integration of autopkgtest runs with the new jenkins.debian.net service ] For discoverability, packages shipping tests for the autopkgtest