Agreed. Let me send this in a final version of this patch. Thanks, Francis
2016-11-08 16:07 GMT-05:00 Mathieu Desnoyers <[email protected] >: > ----- On Nov 8, 2016, at 3:46 PM, Francis Deslauriers < > [email protected]> wrote: > > 2016-11-08 15:06 GMT-05:00 Nathan Lynch <[email protected]>: > >> 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? > > I used grep because the configure script looks for it already > At the moment, we don't check if /bin/true is installed on the machine > using the configure script. > > >> 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. >> > Good point. > I could do: grep "foo" /dev/null > > > utils/utils.sh:45:# We set the default lttng-sessiond path to /bin/true to > prevent the spawning > utils/utils.sh:49:export LTTNG_SESSIOND_PATH="/bin/true" > > already used. > > So unless anyone complain about it, let's use it. > > If someone complains, then we need to fix the configure detection. > > Thanks, > > Mathieu > > > Thank you, > -- > Francis Deslauriers > Software developer > EfficiOS inc. > >> >> _______________________________________________ >> lttng-dev mailing list >> [email protected] >> https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev >> > > > > > > _______________________________________________ > lttng-dev mailing list > [email protected] > https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev > > > -- > Mathieu Desnoyers > EfficiOS Inc. > http://www.efficios.com > -- Francis Deslauriers Software developer EfficiOS inc.
_______________________________________________ lttng-dev mailing list [email protected] https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
