Fix up all use of /dev/stderr for portability to busybox /bin/sh

The typical shell on a small embedded target using busybox does
not have support for /dev/stderr.  A more portable way to send
output to stderr with echo is to redirect stdout to stderr with
1>&2.

In the usttrace script it did something that was effectively
a NOP.  The result of echo "" 2>/dev/stderr is not actually
going to send anything to stderr because the echo is going
to write to stdout.  This case was also fixed.

Signed-off-by: Jason Wessel <jason.wessel@windriver.com>
Signed-off-by: Nils Carlson <nils.carlson@ericsson.com>

diff --git a/tests/test_functions.sh b/tests/test_functions.sh
index 79ade73d6ad1bb840ec6b8f1f9f6fd99e18de053..c4e913bc4345d29d1d4e75e15284a06aedcee3a7 100644 (file)
--- a/tests/test_functions.sh
+++ b/tests/test_functions.sh
@@ -77,7 +77,7 @@ function trace_matches() {
        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
+       echo "$0: No lttv found. Edit \$RUNLTTV to point to your lttv source directory or \$LTTV to you lttv executable." 1>&2
        exit 1;
 
     fi

diff --git a/tests/trace_matches b/tests/trace_matches
index 17a65f8ccf1008a5003483f7af592bced56f122c..da3d95455058b7b489d45382bf07b181b0f9ce94 100755 (executable)
--- a/tests/trace_matches
+++ b/tests/trace_matches
@@ -1,7 +1,7 @@
 #!/bin/bash
 
 function error() {
-       echo "$0: $@" >/dev/stderr
+       echo "$0: $@" 1>&2
 }
 
 function usage() {
@@ -43,7 +43,7 @@ elif [ -x "$(dirname `readlink -f $0`)/../../lttv/runlttv" ]; then
        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
+       echo "$0: No lttv found. Edit \$RUNLTTV to point to your lttv source directory or \$LTTV to you lttv executable." 1>&2
        exit 1;
 
 fi

diff --git a/tests/valgrind_ust-consumerd.sh b/tests/valgrind_ust-consumerd.sh
index 856808bb7d954ac2a5be4a4f64fd1814f76dc124..7dd66681db666dfe689b1f4f72ab842c9a56dd75 100755 (executable)
--- a/tests/valgrind_ust-consumerd.sh
+++ b/tests/valgrind_ust-consumerd.sh
@@ -25,7 +25,7 @@ source $TESTDIR/tap.sh
 starttest "ust-consumerd valgrind check"
 
 if ! which valgrind > /dev/null; then
-    echo "$0: Valgrind not found on the system." >/dev/stderr
+    echo "$0: Valgrind not found on the system." 1>&2
     exit 1;
 fi
 

diff --git a/usttrace b/usttrace
index af80faf76afd3adf7eae740e8f75bc5a910e5abd..e76dae71be1f6590e1ea487bde1ba3c4921ddea3 100755 (executable)
--- a/usttrace
+++ b/usttrace
@@ -4,7 +4,7 @@
 # Distributed under the GPLv2.
 
 error() {
-       echo "$0: error: $1" 2>/dev/stderr
+       echo "$0: error: $1" 1>&2
 }
 
 sighandler() {
@@ -45,18 +45,18 @@ fi
 BASE_TRACE_DIR="${HOME}/.usttraces"
 
 usage() {
-       echo "usage:  $0 OPTIONS COMMAND" 2>/dev/stderr
-       echo "" 2>/dev/stderr
-       echo "Options:" 2>/dev/stderr
-       echo "    -l    Runtime link with UST library." 2>/dev/stderr
-       echo "          (Needed only if program was not linked at compile time with libust.)" 2>/dev/stderr
-       echo "    -L    Add path to ust libraries to LD_LIBRARY_PATH." 2>/dev/stderr
-       echo "    -m    Instrument malloc calls." 2>/dev/stderr
-       echo "    -f    Also trace forked processes." 2>/dev/stderr
-       echo "    -s    Use system-wide daemon instead of creating one for this session." 2>/dev/stderr
-       echo "    -S    Specify the subbuffer size." 2>/dev/stderr
-       echo "    -N    Specify the number of subbuffers." 2>/dev/stderr
-       echo "    -o    Output directory of the trace." 2>/dev/stderr
+       echo "usage:  $0 OPTIONS COMMAND" 1>&2
+       echo "" 1>&2
+       echo "Options:" 1>&2
+       echo "    -l    Runtime link with UST library." 1>&2
+       echo "          (Needed only if program was not linked at compile time with libust.)" 1>&2
+       echo "    -L    Add path to ust libraries to LD_LIBRARY_PATH." 1>&2
+       echo "    -m    Instrument malloc calls." 1>&2
+       echo "    -f    Also trace forked processes." 1>&2
+       echo "    -s    Use system-wide daemon instead of creating one for this session." 1>&2
+       echo "    -S    Specify the subbuffer size." 1>&2
+       echo "    -N    Specify the number of subbuffers." 1>&2
+       echo "    -o    Output directory of the trace." 1>&2
 }
 
 while getopts ":hlLmfsWS:N:o:" options; do