Good catch... My fix was indeed flawed. Acked-by: Yannick Brosseau <[email protected]>
On 2011-02-21 04:19, Nils Carlson wrote: > Signed-off-by: Nils Carlson <[email protected]> > --- > tests/valgrind_ust-consumerd.sh | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/tests/valgrind_ust-consumerd.sh b/tests/valgrind_ust-consumerd.sh > index d89a1f2..1b10772 100755 > --- a/tests/valgrind_ust-consumerd.sh > +++ b/tests/valgrind_ust-consumerd.sh > @@ -17,7 +17,7 @@ > # You should have received a copy of the GNU General Public License > # along with LTTng-UST. If not, see <http://www.gnu.org/licenses/>. > > -if [ -n "$(which valgrind)" ]; then > +if ! which valgrind > /dev/null; then > echo "$0: Valgrind not found on the system." >/dev/stderr > exit 1; > fi > _______________________________________________ ltt-dev mailing list [email protected] http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
