On 11/08/2016 01:49 PM, Francis Deslauriers wrote: > Run a process explicitly in the tracing session to generate the enabled events > rather than relying on the events generated by the lttng CLI.
[...] > diff --git a/tests/regression/kernel/test_all_events > b/tests/regression/kernel/test_all_events > index 50002f4..80a8020 100755 > --- a/tests/regression/kernel/test_all_events > +++ b/tests/regression/kernel/test_all_events > @@ -21,6 +21,8 @@ CURDIR=$(dirname $0)/ > TESTDIR=$CURDIR/../.. > NUM_TESTS=8 > > +TESTCMD="grep -V -q > /dev/null" > + What was wrong with /bin/true, as in v1 of this patch? FYI, Busybox grep doesn't know -V: # grep -V -q >/dev/null grep: invalid option -- 'V' BusyBox v1.24.1 (2016-11-08 12:59:13 CST) multi-call binary. Usage: grep [-HhnlLoqvsriwFE] [-m N] [-A/B/C N] PATTERN/-e PATTERN.../-f FILE [FILE]... # echo $? 1 It looks like the testcases don't care about the exit status of TESTCMD, but you may want to redirect stderr to /dev/null as well. _______________________________________________ lttng-dev mailing list [email protected] https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
