xfs
[Top] [All Lists]

Re: [PATCH V2] xfstests: Fix test wildcard expansion.

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: [PATCH V2] xfstests: Fix test wildcard expansion.
From: Rich Johnston <rjohnston@xxxxxxx>
Date: Fri, 31 May 2013 07:10:10 -0500
Cc: xfstests <xfs@xxxxxxxxxxx>
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20130530223823.GS29466@dastard>
References: <51A7CC6B.3030607@xxxxxxx> <51A7D390.3040108@xxxxxxx> <20130530223823.GS29466@dastard>
User-agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130329 Thunderbird/17.0.5
On 05/30/2013 05:38 PM, Dave Chinner wrote:
On Thu, May 30, 2013 at 05:32:48PM -0500, Rich Johnston wrote:

Before commit 38d58591 "xfstests: fix typo in check",
check xfs/[0-9]?? would execute all tests/xfs/[0-9]?? because:

Change to use egrep to fix the parsing.

Signed-off-by: Rich Johnston <rjohnston@xxxxxxx>
---
V1-V2
Use egrep instead of my bulky method

looks like you lost a bit of the commit message. Otherwise it works
fine here ;)
Oops dang cut/paste, I put it back.

Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>

Thanks for the review Dave.

commit 9267afb16b8e861888e54f829015667753034980
Author: Rich Johnston <rjohnston@xxxxxxx>
Date:   Fri May 31 07:02:48 2013 -0500

    xfstests: Fix test wildcard expansion.

<Prev in Thread] Current Thread [Next in Thread>