All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch@infradead.org>
To: Dave Chinner <david@fromorbit.com>
Cc: xfs@oss.sgi.com
Subject: Re: [PATCH 05/18] xfstests: fold common into check
Date: Tue, 28 Aug 2012 15:52:33 -0400	[thread overview]
Message-ID: <20120828195233.GD15060@infradead.org> (raw)
In-Reply-To: <1343294892-20991-6-git-send-email-david@fromorbit.com>

On Thu, Jul 26, 2012 at 07:27:59PM +1000, Dave Chinner wrote:
> From: Dave Chinner <dchinner@redhat.com>
> 
> There is only one user of the common file now - check. Fold the two
> into one file as the split of functionality is not necessary
> anymore.

Looks good (assuming the bench removal which this probably depends on
goes in)


Reviewed-by: Christoph Hellwig <hch@lst.de>

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

  reply	other threads:[~2012-08-28 19:51 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-26  9:27 [RFC] [PATCH 0/18] xfstests: move tests out of top level Dave Chinner
2012-07-26  9:27 ` [PATCH 01/18] xfstests: remove remake script Dave Chinner
2012-08-28 19:50   ` Christoph Hellwig
2012-07-26  9:27 ` [PATCH 02/18] xfstests: remove bench infrastructure Dave Chinner
2012-07-26  9:27 ` [PATCH 03/18] xfstests: kill useless test owner fields Dave Chinner
2012-08-28 19:51   ` Christoph Hellwig
2012-07-26  9:27 ` [PATCH 04/18] xfstests: remove stale machine configs Dave Chinner
2012-08-28 19:51   ` Christoph Hellwig
2012-07-26  9:27 ` [PATCH 05/18] xfstests: fold common into check Dave Chinner
2012-08-28 19:52   ` Christoph Hellwig [this message]
2012-07-26  9:28 ` [PATCH 06/18] xfstests: clean up and simply check CLI option parsing Dave Chinner
2012-08-28 19:52   ` Christoph Hellwig
2012-07-26  9:28 ` [PATCH 07/18] xfstests: kill hangcheck stuff from check Dave Chinner
2012-08-28 19:53   ` Christoph Hellwig
2012-07-26  9:28 ` [PATCH 08/18] xfstests: remove test expunge file support Dave Chinner
2012-08-28 19:54   ` Christoph Hellwig
2012-07-26  9:28 ` [PATCH 09/18] xfstests: remove undocumented TESTS_REMAINING_LOG Dave Chinner
2012-08-28 19:54   ` Christoph Hellwig
2012-07-26  9:28 ` [PATCH 10/18] xfstests: include test subdirectory support Dave Chinner
2012-07-26  9:28 ` [PATCH 11/18] xfstests: move generic tests out of top level dir Dave Chinner
2012-07-26  9:28 ` [PATCH 12/18] xfstests: move xfs specific tests out of top directory Dave Chinner
2012-07-26  9:28 ` [PATCH 13/18] xfstests: move remaining tests out of top level directory Dave Chinner
2012-07-26  9:28 ` [PATCH 14/18] xfstests: rework CLI individual test specification Dave Chinner
2012-07-26  9:28 ` [PATCH 15/18] xfstests: make exclude groups aware of multiple subdirectories Dave Chinner
2012-07-26  9:28 ` [PATCH 16/18] xfstests: Introduce a results directory Dave Chinner
2012-07-26  9:28 ` [PATCH 17/18] xfstests: convert tests to use new " Dave Chinner
2012-09-05 12:00   ` Boris Ranto
2012-09-05 23:04     ` Dave Chinner
2012-09-06 12:34       ` Boris Ranto
2012-09-06 23:14         ` Dave Chinner
2012-09-07 12:47           ` Boris Ranto
2012-07-26  9:28 ` [PATCH 18/18] xfstests: fix _link_out_file callers Dave Chinner
2012-08-14 21:39 ` [RFC] [PATCH 0/18] xfstests: move tests out of top level Dave Chinner
2012-08-15 17:23   ` Mark Tinguely
2012-08-20 21:27 ` Mark Tinguely
2012-08-20 22:43   ` Dave Chinner
2012-08-21 16:33     ` Ben Myers
2012-08-21 22:09       ` Dave Chinner
2012-08-22 19:16         ` Ben Myers
2012-08-22 23:42           ` Dave Chinner
2012-08-23 17:00             ` Ben Myers
2012-08-24  4:07               ` Dave Chinner
2012-08-28 17:43                 ` Ben Myers
2012-08-28 18:02                   ` Christoph Hellwig
2013-02-25 15:50               ` Eric Sandeen
2013-02-25 21:52                 ` Dave Chinner
2013-02-26  0:27                   ` Theodore Ts'o
2013-02-26  3:18                     ` Dave Chinner
2013-02-26  3:22                     ` Ben Myers

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=20120828195233.GD15060@infradead.org \
    --to=hch@infradead.org \
    --cc=david@fromorbit.com \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.