[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: misc/45038: lack of tests options
The following reply was made to PR misc/45038; it has been noted by GNATS.
From: Nicolas Joly <njoly%pasteur.fr@localhost>
To: Julio Merino <jmmv%NetBSD.org@localhost>
Cc: gnats-bugs%NetBSD.org@localhost, Nicolas Joly <njoly%pasteur.fr@localhost>,
Subject: Re: misc/45038: lack of tests options
Date: Thu, 16 Jun 2011 16:52:50 +0200
On Thu, Jun 16, 2011 at 02:44:21PM +0100, Julio M. Merino Vidal wrote:
> On 6/16/11 2:25 PM, Nicolas Joly wrote:
> > Hi,
> > Does the following patch fit your needs ?
> Looks nice! Some minor comments:
> > Index: share/man/man7/tests.7
> > ===================================================================
> > RCS file: /cvsroot/src/share/man/man7/tests.7,v
> > retrieving revision 1.4
> > diff -u -p -r1.4 tests.7
> > --- share/man/man7/tests.7 7 Nov 2010 17:49:33 -0000 1.4
> > +++ share/man/man7/tests.7 16 Jun 2011 13:19:28 -0000
> Don't forget to bump the date.
Will do. Thanks for the reminder.
> > @@ -163,6 +163,16 @@ The suite-specific configuration file im
> > which contains properties shared among all test suites.
> > These files conform to the configuration file format described in
> > .Xr atf-formats 5 .
> > +.Pp
> > +The following configuration variables are available in the
> > +.Nx
> > +test suite :
> > +.Bl -tag -width "fstype"
> > +.It fstype
> > +When set to a filesystem type, restrict tests programs from the
> > +.Pa /usr/tests/fs/vfs
> I'd add a trailing slash here to denote a directory.
> > +tree to only run test cases for the given type.
> > +.El
> Also document the unprivileged_user option. It's used explicitly by
> some tests. (It defaults to _atf.)
I don't find it anywhere. Is it "require.user" = "unprivileged" ?
IMHO, we should only document in tests(7) the configuration variables
which are not part of ATF.
> > .Ss What to do if something fails?
> > If there is
> > .Em any failure
> Lastly, it may be worth to note that these settings go into
> /etc/atf/NetBSD.conf . Not sure if this is already mentioned in the
It is, a few lines before my patch.
But for `fstype' it should not (at least in most cases) go into
NetBSD.conf; otherwise, you'll always skip most of the vfs tests. It
much more useful to be added to the atf-run command for specific
testing, by example for exercising a specific filesystem when working
Projects and Developments in Bioinformatics
Institut Pasteur, Paris.
Main Index |
Thread Index |