[ltt-dev] [UST PATCH] fix valgrind test valgrind check

Yannick Brosseau yannick.brosseau at gmail.com
Mon Feb 21 16:02:09 EST 2011


Good catch... My  fix was indeed flawed.

Acked-by: Yannick Brosseau <yannick.brosseau at gmail.com>

On 2011-02-21 04:19, Nils Carlson wrote:
> Signed-off-by: Nils Carlson <nils.carlson at ericsson.com>
> ---
>  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
>   





More information about the lttng-dev mailing list