xfs
[Top] [All Lists]

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

To: xfs@xxxxxxxxxxx
Subject: [PATCH 07/18] xfstests: Remove expunge capability for simplicity
From: pwhite@xxxxxxx (Philip White)
Date: Thu, 14 Mar 2013 06:06:11 -0700
Delivered-to: xfs@xxxxxxxxxxx
User-agent: nail 11.25 7/29/05
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

<Prev in Thread] Current Thread [Next in Thread>
  • [PATCH 07/18] xfstests: Remove expunge capability for simplicity, Philip White <=