[ltt-dev] [UST PATCH 1/3] Try harder to find a usable lttv in the tests scripts v2

[yannick.brosseau@gmail.com (Yannick Brosseau)]

Add many more attempts to auto-detect the path to the lttv executable or
the runlttv script in trace comparison tests.
Also support setting the LTTV env variable to directly set a path the
the lttv executable in addition to the RUNLTTV variable

Changelog:
   Correctly manage multiple traces

Signed-off-by: Yannick Brosseau <yannick.brosseau at gmail.com>
---
 tests/test_functions.sh |   46 ++++++++++++++++++++++++++++++++++++++----
 tests/trace_matches     |   50 +++++++++++++++++++++++++++++++++++++++-------
 2 files changed, 83 insertions(+), 13 deletions(-)

diff --git a/tests/test_functions.sh b/tests/test_functions.sh
index 40d4d89..79ade73 100644
--- a/tests/test_functions.sh
+++ b/tests/test_functions.sh
@@ -42,11 +42,44 @@ function check_trace_logs() {
 function trace_matches() {
     local OPTIND=
 
-    RUNLTTV=~/devel/lttv/runlttv
+    #Get a textdump command
+    # if RUNLTTV is defined try to use it
+    # if LTTV variable is defined try to use it
+    # try to find lttv in the path
+    # try to find runlttv in std paths (devel/lttv/runlttv and ust/../lttv/runlttv
 
-    if [ ! -x "$RUNLTTV" ]; then
-	echo "$0: $RUNLTTV not executable. Edit \$RUNLTTV to point to your lttv source directory." >/dev/stderr
+    if [ ! -d "$RUNLTTV" -a -x "$RUNLTTV" ]; then
+	LTTV_TEXTDUMP_CMD="$RUNLTTV -m text "
+	LTTV_TRACE_PREFIX=""
+	
+    elif [ -d "$RUNLTTV" -a -x "$RUNLTTV/runlttv" ]; then 
+	LTTV_TEXTDUMP_CMD="$RUNLTTV/runlttv -m text "
+	LTTV_TRACE_PREFIX=""
+
+    elif [ ! -d "$LTTV" -a -x "$LTTV" ]; then
+	LTTV_TEXTDUMP_CMD="$LTTV -m textDump "
+	LTTV_TRACE_PREFIX="-t"
+
+    elif [ -d "$LTTV" -a -x "$LTTV/lttv" ]; then
+	LTTV_TEXTDUMP_CMD="$LTTV/lttv -m textDump "
+	LTTV_TRACE_PREFIX="-t"
+	
+    elif [ -x "$(which lttv.real)" ]; then
+	LTTV_TEXTDUMP_CMD="$(which lttv.real) -m textDump ";
+	LTTV_TRACE_PREFIX="-t"
+	
+    elif [ -x "~/devel/lttv/runlttv" ]; then
+	LTTV_TEXTDUMP_CMD="~/devel/lttv/runlttv -m text ";
+	LTTV_TRACE_PREFIX=""
+
+    elif [ -x "$(dirname `readlink -f $0`)/../../lttv/runlttv" ]; then
+	LTTV_TEXTDUMP_CMD="$(dirname `readlink -f $0`)/../../lttv/runlttv -m text "
+	LTTV_TRACE_PREFIX=""
+
+    else
+	echo "$0: No lttv found. Edit \$RUNLTTV to point to your lttv source directory or \$LTTV to you lttv executable." >/dev/stderr
 	exit 1;
+
     fi
 
     while getopts ":n:N:" options; do
@@ -71,8 +104,11 @@ function trace_matches() {
 	return 1
     fi
     traces=$(find "$2" -mindepth 1 -maxdepth 1 -type d)
-
-    cnt=$($RUNLTTV -m text "$traces" | grep "$pattern" | wc -l)
+    lttv_trace_cmd=$LTTV_TEXTDUMP_CMD
+    for trace in $traces; do
+	lttv_trace_cmd="$lttv_trace_cmd $LTTV_TRACE_PREFIX $trace"
+    done
+    cnt=$($lttv_trace_cmd | grep "$pattern" | wc -l)
     if [ -z "$expected_count" ]; then
 	if [ "$cnt" -eq "0" ]; then
 	    fail "Did not find at least one instance of $name in trace"
diff --git a/tests/trace_matches b/tests/trace_matches
index ae838fb..17a65f8 100755
--- a/tests/trace_matches
+++ b/tests/trace_matches
@@ -1,9 +1,5 @@
 #!/bin/bash
 
-if [ -z "$RUNLTTV" ]; then
-	RUNLTTV=~/devel/lttv/runlttv
-fi
-
 function error() {
 	echo "$0: $@" >/dev/stderr
 }
@@ -12,9 +8,44 @@ function usage() {
 	echo "Usage: $0 [ -N pattern_name ] [ -n pattern_count ] PATTERN TRACE_PARENT_DIR"
 }
 
-if [ ! -x "$RUNLTTV" ]; then
-	echo "$0: $RUNLTTV not executable. Edit \$RUNLTTV to point to your lttv source directory." >/dev/stderr
+#Get a textdump command
+# if RUNLTTV is defined try to use it
+# if LTTV variable is defined try to use it
+# try to find lttv in the path
+# try to find runlttv in std paths (devel/lttv/runlttv and ust/../lttv/runlttv
+
+if [ ! -d "$RUNLTTV" -a -x "$RUNLTTV" ]; then
+	LTTV_TEXTDUMP_CMD="$RUNLTTV -m text "
+	LTTV_TRACE_PREFIX=""
+
+elif [ -d "$RUNLTTV" -a -x "$RUNLTTV/runlttv" ]; then 
+	LTTV_TEXTDUMP_CMD="$RUNLTTV/runlttv -m text "
+	LTTV_TRACE_PREFIX=""
+
+elif [ ! -d "$LTTV" -a -x "$LTTV" ]; then
+	LTTV_TEXTDUMP_CMD="$LTTV -m textDump "
+	LTTV_TRACE_PREFIX="-t"
+		
+elif [ -d "$LTTV" -a -x "$LTTV/lttv" ]; then
+	LTTV_TEXTDUMP_CMD="$LTTV/lttv -m textDump "
+	LTTV_TRACE_PREFIX="-t"
+
+elif [ -x "$(which lttv.real)" ]; then
+	LTTV_TEXTDUMP_CMD="$(which lttv.real) -m textDump ";
+	LTTV_TRACE_PREFIX="-t"
+
+elif [ -x "~/devel/lttv/runlttv" ]; then
+	LTTV_TEXTDUMP_CMD="~/devel/lttv/runlttv -m text ";
+	LTTV_TRACE_PREFIX=""
+
+elif [ -x "$(dirname `readlink -f $0`)/../../lttv/runlttv" ]; then
+	LTTV_TEXTDUMP_CMD="$(dirname `readlink -f $0`)/../../lttv/runlttv -m text "
+	LTTV_TRACE_PREFIX=""
+
+else
+	echo "$0: No lttv found. Edit \$RUNLTTV to point to your lttv source directory or \$LTTV to you lttv executable." >/dev/stderr
 	exit 1;
+
 fi
 
 while getopts ":n:N:" options; do
@@ -44,10 +75,13 @@ if [ -z "$2" ]; then
 	exit 1
 fi
 traces=$(find "$2" -mindepth 1 -maxdepth 1 -type d)
-
+lttv_trace_cmd=$LTTV_TEXTDUMP_CMD
+for trace in $traces; do
+	lttv_trace_cmd="$lttv_trace_cmd $LTTV_TRACE_PREFIX $trace"
+done
 echo -n "Analyzing trace ($name): "
 
-cnt=$($RUNLTTV -m text "$traces" | grep "$pattern" | wc -l)
+cnt=$($lttv_trace_cmd | grep "$pattern" | wc -l)
 if [ -z "$expected_count" ]; then
 	if [ "$cnt" -eq "0" ]; then
 		echo "ERROR"
-- 
1.7.2.3