[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

implicit debian/tests/control files

On Fri, Sep 12, 2014 at 07:22:46AM +0200, Martin Pitt wrote:
> Niko Tyni [2014-09-11 22:55 +0300]:
> > > Admittedly these dynamically generated control files are a bit against
> > > the spirit of DEP-8 to have an explicit test control. However, I think
> > > it's not too bad as the benefit of not having to change 3.000 source
> > > packages all the time is just immense. The one thing I'd like to add
> > > to the source packages is an "XS-Testsuite: autopkgtest" field though,
> > > so that we can stop maintaining hardcoded whitelists at some point
> > > (but from my POV it's ok if that takes several years even).
> > 
> > While I certainly agree about the benefits, I'm worried about other
> > consumers of debian/tests/control, which would have to reimplement the
> > implicit control files.  There's at least sadt(1) in devscripts that I
> > know of. 
> Ah, right. From my side I'd certainly like to make this as practical
> and useful to *you* as the consumers of this. But yeah, sometimes I
> keep forgetting that adt-run isn't the only implementation of this.

We could extract this logic into a dependency package that both
autopkgtest and sadt could use, as well as any other implementation that
comes up.

> > However, I'm somewhat against including the Testsuite headers without a
> > debian/tests/control file, as that could actually break other (current
> > or future) implementations.
> I think one way to get that "mass-run" effect without breaking the
> spec would be to use a different XS-Testsuite: value. How about
> "autopkgtest-auto-perl"? We'd use something similar for
> "autopkgtest-auto-ruby". Test runners which use another implementation
> which doesn't provide these would then just not run Perl/Ruby
> packages, ci.d.n. could just watch out for the additional tags once it
> updates to autopkgtest 3.5, and we satisfy the spec that
> "Testsuite: autopkgtest" packages need to have a debian/tests/control.

That makes sense to me.

Antonio Terceiro <terceiro at debian.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/autopkgtest-devel/attachments/20140912/14fe102a/attachment-0001.sig>

Reply to: