On Thu, 6 Jul 2017 09:50:26 -0400 Steven Rostedt <rost...@goodmis.org> wrote:
> On Thu, 6 Jul 2017 18:10:58 +0900 > Masami Hiramatsu <mhira...@kernel.org> 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 <mhira...@kernel.org>