On Mon, 28 May 2018 01:32:58 +0900 Masami Hiramatsu <mhiramat@xxxxxxxxxx> wrote: > Hi Steve, > > On Fri, 25 May 2018 17:13:53 -0400 > Steven Rostedt <rostedt@xxxxxxxxxxx> wrote: > > > On Fri, 25 May 2018 17:12:29 -0400 > > Steven Rostedt <rostedt@xxxxxxxxxxx> wrote: > > > > > > > #!/bin/sh > > > > Hmm, I think I need to make this #!/bin/bash > > > > > test_trace() { > > > file=$1 > > > x=$2 > > > > > > cat $file | while read line; do > > > if [ "$line" != "${line/\#/}" ]; then > > > continue > > > fi > > > echo "testing $line for >$x<" > > > if [ "$line" == "${line/>$x</}" ]; then > > > > The ${line/>$x</} is unique to bash I believe. > > Hmm, could you try to use only posix-shell based test ? > Since I would like to keep this can run on small > environment, like busybox, etc. > Like something like this? -- Steve diff --git a/tools/testing/selftests/ftrace/test.d/trigger/trigger-trace-marker-snapshot.tc b/tools/testing/selftests/ftrace/test.d/trigger/trigger-trace-marker-snapshot.tc index c6c50a2..3a3c33d 100644 --- a/tools/testing/selftests/ftrace/test.d/trigger/trigger-trace-marker-snapshot.tc +++ b/tools/testing/selftests/ftrace/test.d/trigger/trigger-trace-marker-snapshot.tc @@ -41,11 +41,13 @@ test_trace() { x=$2 cat $file | while read line; do - if [ "$line" != "${line/\#/}" ]; then + comment=`echo $line | sed -e 's/^#//'` + if [ "$line" != "$comment" ]; then continue fi echo "testing $line for >$x<" - if [ "$line" == "${line/>$x</}" ]; then + match=`echo $line | sed -e "s/>$x<//"` + if [ "$line" == "$match" ]; then fail "$line does not have >$x< in it" fi let x=$x+2 -- To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html