From: Theodore Ts'o <tytso@mit.edu>
To: Dave Chinner <david@fromorbit.com>
Cc: xfs@oss.sgi.com, Ext4 Developers List <linux-ext4@vger.kernel.org>
Subject: Re: [PATCH -v2] check: add support for an external file containing tests to exclude
Date: Mon, 12 May 2014 08:42:32 -0400 [thread overview]
Message-ID: <20140512124232.GB31376@thunk.org> (raw)
In-Reply-To: <20140416064018.GJ15995@dastard>
On Wed, Apr 16, 2014 at 04:40:18PM +1000, Dave Chinner wrote:
> On Wed, Apr 16, 2014 at 12:58:22AM -0400, Theodore Ts'o wrote:
> > Currently the -X option is intended to specify a set of expunging
> > files which are stored in each test/* subdirectory. As described in
> > the commit description for 0b1e8abd4, in order to exclude the test
> > generic/280, the -X option is used as follows:
> >
> > $ cat tests/generic/3.0-stable-avoid
> > 280
> > $ sudo ./check -X 3.0-stable-avoid generic/280
> >
> > However, it is sometimes useful to store the set of expunged tests in
> > a single file, outside of tests/* subdirectories. This commit enables
> > the following:
> >
> > $ cat /root/conf/data_journal.exclude
> > generic/068
> > ext4/301
> > $ sudo ./check -E /root/conf/data_journal.exclude -g auto
> >
> > Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
>
> Looks good.
>
> Reviewed-by: Dave Chinner <dchinner@redhat.com>
This patch has apparently not hit the xfstests repository. Would you
like me to resend it?
Thanks,
- Ted
next prev parent reply other threads:[~2014-05-12 12:42 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-15 2:52 [PATCH] check: add support for an external test expunging file Theodore Ts'o
2014-04-16 0:15 ` Dave Chinner
2014-04-16 0:25 ` Theodore Ts'o
2014-04-16 0:34 ` Dave Chinner
2014-04-16 4:58 ` [PATCH -v2] check: add support for an external file containing tests to exclude Theodore Ts'o
2014-04-16 6:40 ` Dave Chinner
2014-05-12 12:42 ` Theodore Ts'o [this message]
2014-05-12 22:00 ` Dave Chinner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140512124232.GB31376@thunk.org \
--to=tytso@mit.edu \
--cc=david@fromorbit.com \
--cc=linux-ext4@vger.kernel.org \
--cc=xfs@oss.sgi.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).