On 8/8/13 3:17 AM, Jan Schmidt wrote:
> These two patches add the announced tests for btrfs send / receive. As
> requested, the fssum tool is now included.
> One drawback is that I'm unable to edit configure.ac or whatever needs
> to be modified in an autotools preferred way. Any hints appreciated,
> preferrably hints containing all the modifications required to introduce
> something like HAVE_SEEK_HOLE.
Other tests in the tree just add:
#define SEEK_DATA 3
#define SEEK_HOLE 4
> I do not want to make modifications to fssum.c here, if that's
> absolutely required (because one /could/ get along using linux/fs.h,
> which is not the way I would like to go), I'd like to have that changed
> in the far-progs repository where fssum.c comes from as well.
Well, unfortunately it breaks the build w/o some change or other,
on older distros:
fssum.c: In function 'sum_file_data_permissive':
fssum.c:243: error: 'SEEK_DATA' undeclared (first use in this function)
so this can't be merged as-is.
Adding the 4 lines above to the xfstests copy seems like a pretty obvious fix
to get the tool building and move this along.
Then, to simply skip this test if the kernel we're running on doesn't grok
to your new test in patch 2.
> - included fssum
> - test number is now 316 (was 314)
> - 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)
> Jan Schmidt (2):
> xfstests: add fssum tool
> xfstests btrfs/316: test send / receive
> .gitignore | 1 +
> common/config | 2 +
> src/Makefile | 11 +-
> src/fssum.c | 819
> tests/btrfs/316 | 113 +++++++
> tests/btrfs/316.out | 4 +
> tests/btrfs/group | 1 +
> 7 files changed, 950 insertions(+), 1 deletions(-)
> create mode 100644 src/fssum.c
> create mode 100755 tests/btrfs/316
> create mode 100644 tests/btrfs/316.out