[PATCH] xfstests: make sure $seq.full files don't land in /

[PATCH] xfstests: make sure $seq.full files don't land in /

[Eric Sandeen]

186 and 192, at least, were landing (part of) their $seq.full
files in /

Fix this by using $here/$seq.full where needed.

Signed-off-by: Eric Sandeen <sandeen@sandeen.net>
---

diff --git a/186 b/186
index 94d399a..643f4b1 100755
--- a/186
+++ b/186
@@ -79,7 +79,7 @@ _rmv_eas()
 
 _filter_inode()
 {
-	tee -a $seq.full | \
+	tee -a $here/$seq.full | \
 		sed -e "s/core.forkoff/forkoff/g" | \
 		egrep '^u.sfdir2|^a.sfattr|forkoff' | \
 		egrep -v 'inumber|parent'
@@ -87,7 +87,7 @@ _filter_inode()
 
 _filter_version()
 {
-	tee -a $seq.full | tr ',' '\n' | grep ATTR
+	tee -a $here/$seq.full | tr ',' '\n' | grep ATTR
 }
 
 _print_inode()
diff --git a/192 b/192
index 0d0b294..d8301d5 100755
--- a/192
+++ b/192
@@ -68,7 +68,7 @@ time2=`_access_time $testfile | tee -a $seq.full`
 cd /
 umount $TEST_DIR
 _test_mount
-time3=`_access_time $testfile | tee -a $seq.full`
+time3=`_access_time $testfile | tee -a $here/$seq.full`
 
 delta1=`expr $time2 - $time1`
 delta2=`expr $time3 - $time1`


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

Re: [PATCH] xfstests: make sure $seq.full files don't land in /

[Christoph Hellwig]

On Wed, Apr 14, 2010 at 11:31:34AM -0500, Eric Sandeen wrote:
> 186 and 192, at least, were landing (part of) their $seq.full
> files in /
> 
> Fix this by using $here/$seq.full where needed.
> 
> Signed-off-by: Eric Sandeen <sandeen@sandeen.net>

Looks good,


Reviewed-by: Christoph Hellwig <hch@lst.de>

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