make sure to _require_attrs when necessary
PATCH 1/1 make sure to _require_attrs when necessary
xfs/073 and xfs/295 forgot to _require_attrs. Fix it.
Signed-off-by Jeff Moyer <[email protected]>
tests/xfs/073 +2 -0
@@ -33,6 +33,7 @@ status=1 # failure is the default! # get standard environment, filters and checks . ./common/rc . ./common/filter +. ./common/attr # don't put fs images in /tmp imgs=$TEST_DIR/$$
@@ -127,6 +128,7 @@ _verify_copy() _supported_fs xfs _supported_os Linux _require_test +_require_attrs [ "$USE_EXTERNAL" = yes ] && _notrun "Cannot xfs_copy with external devices" [ -n "$XFS_COPY_PROG" ] || _notrun "xfs_copy binary not yet installed"
tests/xfs/295 +2 -0
@@ -39,6 +39,7 @@ _cleanup() # get standard environment, filters and checks . ./common/rc . ./common/filter +. ./common/attr # real QA test starts here
@@ -46,6 +47,7 @@ _cleanup() _supported_fs xfs _supported_os IRIX Linux _require_scratch +_require_attrs rm -f $seqres.full
Quoted 1 line · Jeff Moyer
xfs/073 and xfs/295 forgot to
_require_attrs. Fix it.
Reviewed-by Eric Sandeen <[email protected]>
thanks!
Quoted 47 lines
Signed-off-by Jeff Moyer <[email protected]>tests/xfs/073 +2
-0@@ -33,6 +33,7 @@ status=1 # failure is the default! # get standard environment, filters and checks . ./common/rc . ./common/filter +. ./common/attr # don't put fs images in /tmp imgs=$TEST_DIR/$$@@ -127,6 +128,7 @@ _verify_copy() _supported_fs xfs _supported_os Linux _require_test +_require_attrs [ "$USE_EXTERNAL" = yes ] && _notrun "Cannot xfs_copy with external devices" [ -n "$XFS_COPY_PROG" ] || _notrun "xfs_copy binary not yet installed"tests/xfs/295 +2
-1@@ -39,6 +39,7 @@ _cleanup() # get standard environment, filters and checks . ./common/rc . ./common/filter +. ./common/attr # real QA test starts here@@ -46,6 +47,7 @@ _cleanup() _supported_fs xfs _supported_os IRIX Linux _require_scratch +_require_attrs rm -f $seqres.full --To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html