Re: [PATCH v3 4/5] selftests: ftrace: Output only to console with "--logdir -"

From: Masami Hiramatsu
Date: Thu Jul 06 2017 - 20:43:37 EST


On Thu, 6 Jul 2017 09:50:26 -0400
Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:

> On Thu, 6 Jul 2017 18:10:58 +0900
> Masami Hiramatsu <mhiramat@xxxxxxxxxx> wrote:
>
> > @@ -255,12 +262,18 @@ __run_test() { # testfile
> > # Run one test case
> > run_test() { # testfile
> > local testname=`basename $1`
> > - local testlog=`mktemp $LOG_DIR/${testname}-log.XXXXXX`
> > + if [ ! -z "$LOG_FILE" ] ; then
> > + local testlog=`mktemp $LOG_DIR/${testname}-log.XXXXXX`
> > + else
> > + local testlog=`/proc/self/fd/1`
>
> As Stafford mentioned. I don't think you want backticks here.

Oops, right!

--
Masami Hiramatsu <mhiramat@xxxxxxxxxx>