[PATCH] xfstests: Fix test wildcard expansion.

Dave Chinner david at fromorbit.com
Thu May 30 17:12:11 CDT 2013


On Thu, May 30, 2013 at 05:02:19PM -0500, Rich Johnston wrote:
> 
> Before commit 38d58591 "xfstests: fix typo in check",
> check xfs/[0-9]?? would execute all tests/xfs/[0-9]?? because:
> 
> 'if grep "^$testname" $group_file >/dev/null'
> returns the contents of $group_file because $testname="".
> 
> Therefore xfs/[0-9]?? was echoed to $tmp.list
> 
> Fix the parsing in check to expand the regular expressions for test names.

I just fixed this locally myself:

-                       if grep "^$test_name" $group_file >/dev/null ; then
+                       if egrep "^$test_name" $group_file >/dev/null ; then

Which just tells grep to treat the pattern as an extended regex....

Cheers,

Dave.
-- 
Dave Chinner
david at fromorbit.com



More information about the xfs mailing list