xfs
[Top] [All Lists]

Re: [PATCH] xfstests: filter out latest xfsdump version messages

To: Brian Foster <bfoster@xxxxxxxxxx>
Subject: Re: [PATCH] xfstests: filter out latest xfsdump version messages
From: Eric Sandeen <sandeen@xxxxxxxxxx>
Date: Mon, 18 Jun 2012 11:18:03 -0500
Cc: xfs@xxxxxxxxxxx
In-reply-to: <1340034963-24932-1-git-send-email-bfoster@xxxxxxxxxx>
References: <1340034963-24932-1-git-send-email-bfoster@xxxxxxxxxx>
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:12.0) Gecko/20120428 Thunderbird/12.0.1
On 6/18/12 10:56 AM, Brian Foster wrote:
> The latest version of xfsdump/restore (3.1.0) causes dump tests
> to complain. Fix up the filter to include a wider selection of
> version numbers.

Looks good to me, thanks - I'll merge it, but I think I'll
add a "^version" to it to make it a little more selective,
if that's ok with you.

Reviewed-by: Eric Sandeen <sandeen@xxxxxxxxxx>

> Signed-off-by: Brian Foster <bfoster@xxxxxxxxxx>
> ---
>  common.dump |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/common.dump b/common.dump
> index ca09ec1..61d3f43 100644
> --- a/common.dump
> +++ b/common.dump
> @@ -863,7 +863,7 @@ _dump_filter_main()
>        -e 's/1048576/BLOCKSZ/'                                \
>        -e 's/2097152/BLOCKSZ/'                                \
>        -e 's/(pid[     ]*[1-9][0-9]*)/\(pid PID\)/'   \
> -      -e '/version 3\.0/d'                           \
> +      -e '/version [3-9]\.[0-9]/d'                           \
>        -e 's/\/hw\/module.*$/SCRATCH_DEV/'            \
>        -e 's/xfsdump: ino map phase 1: .*/xfsdump: ino map <PHASES>/' \
>        -e '/xfsdump: ino map phase [2]/,1d'           \

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