On Wed, Oct 01, 2014 at 05:27:18PM +0200, Jan Kara wrote:
> Use appropriate environment variable (XFSDUMP_PROG) instead of hardcoding
> 'xfsdump' name. This also makes the test fail graciously when xfsdump
> isn't installed.
>
> Signed-off-by: Jan Kara <jack@xxxxxxx>
> ---
> tests/xfs/195 | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/tests/xfs/195 b/tests/xfs/195
> index c7bc7b8..b86aedb 100755
> --- a/tests/xfs/195
> +++ b/tests/xfs/195
> @@ -48,7 +48,7 @@ trap "_cleanup; exit \$status" 0 1 2 3 15
> #
> _do_dump()
> {
> - xfsdump -l 0 -s d -F \
> + $XFSDUMP_PROG -l 0 -s d -F \
> -L prova -M prova \
> -f $TEST_DIR/dumpfile -e -v excluded_files=debug $TEST_DIR \
> | grep "ino $inum" \
> @@ -58,6 +58,7 @@ _do_dump()
> # get standard environment, filters and checks
> . ./common/rc
> . ./common/filter
> +. ./common/dump
This causes the test to fail, because _do_dump() is defined in
common/dump as well. I'll fix the conflict...
Cheers,
Dave.
>
> # real QA test starts here
> _supported_fs xfs
> --
> 1.6.0.2
>
> --
> To unsubscribe from this list: send the line "unsubscribe fstests" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
Dave Chinner
david@xxxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe fstests" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
