| Message ID | 1433272721-24871-2-git-send-email-jsnow@redhat.com |
|---|---|
| State | New |
| Headers | show |
On Tue, 06/02 15:18, John Snow wrote: > If you are running out-of-tree, the -x option to exclude > a certain iotest is broken. > > Replace porcelain usage of ls with a sturdier awk command. > > Signed-off-by: John Snow <jsnow@redhat.com> > --- Reviewed-by: Fam Zheng <famz@redhat.com> > tests/qemu-iotests/common | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/tests/qemu-iotests/common b/tests/qemu-iotests/common > index 1030aaf..84b6f16 100644 > --- a/tests/qemu-iotests/common > +++ b/tests/qemu-iotests/common > @@ -83,7 +83,8 @@ s/ .*//p > elif $xgroup > then > # arg after -x > - [ ! -s $tmp.list ] && ls [0-9][0-9][0-9] [0-9][0-9][0-9][0-9] >$tmp.list 2>/dev/null > + # Populate $tmp.list with all tests > + awk '/^[0-9]{3,}/ {print $1}' "${source_iotests}/group" > $tmp.list 2>/dev/null > group_list=`sed -n <"$source_iotests/group" -e 's/$/ /' -e "/^[0-9][0-9][0-9].* $r /"'{ > s/ .*//p > }'` > -- > 2.1.0 >
diff --git a/tests/qemu-iotests/common b/tests/qemu-iotests/common index 1030aaf..84b6f16 100644 --- a/tests/qemu-iotests/common +++ b/tests/qemu-iotests/common @@ -83,7 +83,8 @@ s/ .*//p elif $xgroup then # arg after -x - [ ! -s $tmp.list ] && ls [0-9][0-9][0-9] [0-9][0-9][0-9][0-9] >$tmp.list 2>/dev/null + # Populate $tmp.list with all tests + awk '/^[0-9]{3,}/ {print $1}' "${source_iotests}/group" > $tmp.list 2>/dev/null group_list=`sed -n <"$source_iotests/group" -e 's/$/ /' -e "/^[0-9][0-9][0-9].* $r /"'{ s/ .*//p }'`
If you are running out-of-tree, the -x option to exclude a certain iotest is broken. Replace porcelain usage of ls with a sturdier awk command. Signed-off-by: John Snow <jsnow@redhat.com> --- tests/qemu-iotests/common | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)