[lttng-dev] [PATCH lttng-tools] Fix: test: use load_path instead of $1

Jérémie Galarneau jeremie.galarneau at efficios.com
Mon Jan 25 16:12:08 EST 2016


Merged, thanks!

Jérémie

On Thu, Jan 14, 2016 at 1:40 PM, Jonathan Rajotte
<jonathan.rajotte-julien at efficios.com> wrote:
> $1 is used for tap condition checking not the load path.
>
> Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien at efficios.com>
> ---
>  tests/utils/utils.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/utils/utils.sh b/tests/utils/utils.sh
> index 3520792..42b3441 100644
> --- a/tests/utils/utils.sh
> +++ b/tests/utils/utils.sh
> @@ -362,7 +362,7 @@ function start_lttng_sessiond_opt()
>         if [ -z $(pgrep --full lt-$SESSIOND_BIN) ]; then
>                 # Have a load path ?
>                 if [ -n "$load_path" ]; then
> -                       $DIR/../src/bin/lttng-sessiond/$SESSIOND_BIN --load "$1" --background --consumerd32-path="$DIR/../src/bin/lttng-consumerd/lttng-consumerd" --consumerd64-path="$DIR/../src/bin/lttng-consumerd/lttng-consumerd"
> +                       $DIR/../src/bin/lttng-sessiond/$SESSIOND_BIN --load "$load_path" --background --consumerd32-path="$DIR/../src/bin/lttng-consumerd/lttng-consumerd" --consumerd64-path="$DIR/../src/bin/lttng-consumerd/lttng-consumerd"
>                 else
>                         $DIR/../src/bin/lttng-sessiond/$SESSIOND_BIN --background --consumerd32-path="$DIR/../src/bin/lttng-consumerd/lttng-consumerd" --consumerd64-path="$DIR/../src/bin/lttng-consumerd/lttng-consumerd"
>                 fi
> --
> 2.5.0
>



-- 
Jérémie Galarneau
EfficiOS Inc.
http://www.efficios.com



More information about the lttng-dev mailing list