All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Sandeen <sandeen@sandeen.net>
To: Jan Schmidt <list.xfs@jan-o-sch.net>
Cc: xfs@oss.sgi.com, linux-btrfs@vger.kernel.org, sbehrens@giantdisaster.de
Subject: Re: [PATCH v4 0/2] xfstest btrfs/316: test send / receive
Date: Tue, 13 Aug 2013 13:33:16 -0500	[thread overview]
Message-ID: <520A7BEC.3010301@sandeen.net> (raw)
In-Reply-To: <1376414658-4563-1-git-send-email-list.xfs@jan-o-sch.net>

On 8/13/13 12:24 PM, Jan Schmidt wrote:
> These two patches add the announced tests for btrfs send / receive. As
> requested, the fssum tool is now included.


Thanks for the updates.

Both:
Reviewed-by: Eric Sandeen <sandeen@redhat.com>

> --
> v1->v2:
>  - included fssum
>  - test number is now 316 (was 314)
> v2->v3:
>  - added missing -lcrypto to build fssum
>  - removed obsolete change in README now that fssum is included
>  - fixed comment in test/btrfs/316's header (314 -> 316)
> v3->v4:
>  - build fssum with help of autotools only if libssl is available
>  - removed clumsy OPT_TARGETS in src/Makefile
>  - added #define directives for SEEK_DATA and SEEK_HOLE to fssum.c
> 
> Jan Schmidt (2):
>   xfstests: add fssum tool
>   xfstests btrfs/316: test send / receive
> 
>  .gitignore           |    1 +
>  aclocal.m4           |    1 +
>  configure.ac         |    1 +
>  include/builddefs.in |    1 +
>  m4/Makefile          |    1 +
>  m4/package_ssldev.m4 |    4 +
>  src/Makefile         |    8 +
>  src/fssum.c          |  828 ++++++++++++++++++++++++++++++++++++++++++++++++++
>  tests/btrfs/316      |  116 +++++++
>  tests/btrfs/316.out  |    4 +
>  tests/btrfs/group    |    1 +
>  11 files changed, 966 insertions(+), 0 deletions(-)
>  create mode 100644 m4/package_ssldev.m4
>  create mode 100644 src/fssum.c
>  create mode 100755 tests/btrfs/316
>  create mode 100644 tests/btrfs/316.out
> 


WARNING: multiple messages have this Message-ID (diff)
From: Eric Sandeen <sandeen@sandeen.net>
To: Jan Schmidt <list.xfs@jan-o-sch.net>
Cc: sbehrens@giantdisaster.de, linux-btrfs@vger.kernel.org, xfs@oss.sgi.com
Subject: Re: [PATCH v4 0/2] xfstest btrfs/316: test send / receive
Date: Tue, 13 Aug 2013 13:33:16 -0500	[thread overview]
Message-ID: <520A7BEC.3010301@sandeen.net> (raw)
In-Reply-To: <1376414658-4563-1-git-send-email-list.xfs@jan-o-sch.net>

On 8/13/13 12:24 PM, Jan Schmidt wrote:
> These two patches add the announced tests for btrfs send / receive. As
> requested, the fssum tool is now included.


Thanks for the updates.

Both:
Reviewed-by: Eric Sandeen <sandeen@redhat.com>

> --
> v1->v2:
>  - included fssum
>  - test number is now 316 (was 314)
> v2->v3:
>  - added missing -lcrypto to build fssum
>  - removed obsolete change in README now that fssum is included
>  - fixed comment in test/btrfs/316's header (314 -> 316)
> v3->v4:
>  - build fssum with help of autotools only if libssl is available
>  - removed clumsy OPT_TARGETS in src/Makefile
>  - added #define directives for SEEK_DATA and SEEK_HOLE to fssum.c
> 
> Jan Schmidt (2):
>   xfstests: add fssum tool
>   xfstests btrfs/316: test send / receive
> 
>  .gitignore           |    1 +
>  aclocal.m4           |    1 +
>  configure.ac         |    1 +
>  include/builddefs.in |    1 +
>  m4/Makefile          |    1 +
>  m4/package_ssldev.m4 |    4 +
>  src/Makefile         |    8 +
>  src/fssum.c          |  828 ++++++++++++++++++++++++++++++++++++++++++++++++++
>  tests/btrfs/316      |  116 +++++++
>  tests/btrfs/316.out  |    4 +
>  tests/btrfs/group    |    1 +
>  11 files changed, 966 insertions(+), 0 deletions(-)
>  create mode 100644 m4/package_ssldev.m4
>  create mode 100644 src/fssum.c
>  create mode 100755 tests/btrfs/316
>  create mode 100644 tests/btrfs/316.out
> 

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

  parent reply	other threads:[~2013-08-13 18:33 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-13 17:24 [PATCH v4 0/2] xfstest btrfs/316: test send / receive Jan Schmidt
2013-08-13 17:24 ` Jan Schmidt
2013-08-13 17:24 ` [PATCH v4 1/2] xfstests: add fssum tool Jan Schmidt
2013-08-13 17:24   ` Jan Schmidt
2013-08-13 17:24 ` [PATCH v4 2/2] xfstests btrfs/316: test send / receive Jan Schmidt
2013-08-13 17:24   ` Jan Schmidt
2013-08-13 18:33 ` Eric Sandeen [this message]
2013-08-13 18:33   ` [PATCH v4 0/2] xfstest " Eric Sandeen
2013-08-13 22:31   ` Rich Johnston
2013-08-13 22:31     ` Rich Johnston

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=520A7BEC.3010301@sandeen.net \
    --to=sandeen@sandeen.net \
    --cc=linux-btrfs@vger.kernel.org \
    --cc=list.xfs@jan-o-sch.net \
    --cc=sbehrens@giantdisaster.de \
    --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.