[PATCH] xfstests: move _filter_ln to common.filter

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

 



Move test 103's _filter_ln to common.filter and make
it more generic (not depending on 103's pathnames).

TBH I've lost my children's treasury of ln failure
messages, so I'm not sure this catches all variants;
it's hard to work backwards from the existing sed script
to what the various outputs were.  This works for me
but might need more tweaking on other systems.

Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx>
---

diff --git a/103 b/103
index e383991..a390739 100755
--- a/103
+++ b/103
@@ -54,13 +54,6 @@ _create_scratch()
 	fi
 }
 
-_filter_ln()
-{
-    sed -e "s,SCRATCH_MNT/nosymlink/target - Operation not permitted,ln: creating symbolic link \`SCRATCH_MNT/nosymlink/target\' to \`SCRATCH_MNT/nosymlink/source\': Operation not permitted,g" \
-        -e "/SCRATCH_MNT.*to.*SCRATCH_MNT/! s,: Operation not permitted, to \`SCRATCH_MNT/nosymlink/source\': Operation not permitted,g" \
-	-e "s,failed to create,creating,"
-}
-
 _filter_noymlinks_flag()
 {
 	_test_inode_flag nosymlinks $SCRATCH_MNT/nosymlink
diff --git a/103.out b/103.out
index 45c0aa8..f939bcd 100644
--- a/103.out
+++ b/103.out
@@ -4,7 +4,7 @@ QA output created by 103
 *** testing nosymlinks directories
 *** setting nosymlinks bit
 --n-- SCRATCH_MNT/nosymlink
-ln: creating symbolic link `SCRATCH_MNT/nosymlink/target' to `SCRATCH_MNT/nosymlink/source': Operation not permitted
+ln: creating symbolic link `SCRATCH_MNT/nosymlink/target': Operation not permitted
 *** 1st listing...
 SCRATCH_MNT
 SCRATCH_MNT/nosymlink
diff --git a/common.filter b/common.filter
index f0f6076..9e4c90c 100644
--- a/common.filter
+++ b/common.filter
@@ -229,5 +229,12 @@ _filter_spaces()
        sed -e 's/ [ ]*/ /g'
 }
 
+# Account for different "ln" failure messages
+_filter_ln()
+{
+	sed -e "s,\(creating symbolic link .*\) to .*: ,\1," \
+	    -e "s,failed to create,creating,"
+}
+
 # make sure this script returns success
 /bin/true


_______________________________________________
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