[PATCH 07/18] xfstests: Remove expunge capability for simplicity

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Phil White <pwhite@xxxxxxx>


This is a rebasing & resubmit of a dchinner patch.  His comments on the
original:
-----------------
Check supports an expunge file for tests - a way of marking tests
not to run even if you specify it to run. Use is like this:

$ echo 002 > expunged
test-2:~/src/xfstests-dev$ sudo ./check 001-003
FSTYP         -- xfs (debug)
PLATFORM      -- Linux/x86_64 test-2 3.5.0-rc1-dgc+
MKFS_OPTIONS  -- -f -bsize=4096 /dev/vdb
MOUNT_OPTIONS -- /dev/vdb /mnt/scratch

001 4s ... 4s
002 - expunged
003 0s ... 0s
Ran: 001 002 003
Not run:1
Passed all 2 tests

This is arguably useful(*), but gets in the way of splitting up the
tests into multiple directories. Remove it for now, but such
functionality should be considered for re-implementation at a later
date.

(*) e.g. having a different "expunged" file for each distro release
you have to test to avoid the tests known to fail due to fixes or
features that will never be back ported to older releases....
-----------------

Signed-off-by: Phil White <pwhite@xxxxxxx>
---
 check  |   13 -------------
 common |    5 -----
 2 files changed, 0 insertions(+), 18 deletions(-)

diff --git a/check b/check
index e2c758b..06189c8 100755
--- a/check
+++ b/check
@@ -95,13 +95,6 @@ END	{ if (NR > 0) {
 	    mv $tmp.out check.time
 	fi
 
-	if [ -f $tmp.expunged ]
-	then
-	    notrun=`wc -l <$tmp.expunged | sed -e 's/  *//g'`
-	    n_try=`expr $n_try - $notrun`
-	    list=`echo "$list" | sed -f $tmp.expunged`
-	fi
-
 	echo "" >>check.log
 	date >>check.log
 	echo $list | fmt | sed -e 's/^/    /' >>check.log
@@ -195,15 +188,9 @@ do
     then
 	echo
 	continue
-    elif [ -f expunged ] && $expunge && egrep "^$seq([ 	]|\$)" expunged >/dev/null
-    then
-	echo " - expunged"
-	rm -f $seq.out.bad
-	echo "/^$seq\$/d" >>$tmp.expunged
     elif [ ! -f $seq ]
     then
 	echo " - no such test?"
-	echo "/^$seq\$/d" >>$tmp.expunged
     else
 	# really going to try and run this one
 	#
diff --git a/common b/common
index 2553fe9..821ea59 100644
--- a/common
+++ b/common
@@ -49,10 +49,6 @@ BEGIN  { for (t='$start'; t<='$end'; t++) printf "%03d\n",t }' \
 		if grep -s "^$id " group >/dev/null ; then
 			# in group file ... OK
 			echo $id >>$tmp.list
-		elif [ -f expunged ] && $expunge && \
-				egrep "^$id([   ]|\$)" expunged >/dev/null; then
-			# expunged ... will be reported, but not run, later
-			echo $id >>$tmp.list
 		else
 			# oops
 			echo "$id - unknown test, ignored"
@@ -81,7 +77,6 @@ group=false
 xgroup=false
 showme=false
 sortme=false
-expunge=true
 have_test_arg=false
 randomize=false
 timestamp=${TIMESTAMP:=false}
-- 
1.5.3.8

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


[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux