[lttng-dev] [PATCH lttng-tools] Fix: test: append to LD_LIBRARY_PATH

[Jérémie Galarneau]

Merged, thanks!

Jérémie

On Thu, Jan 14, 2016 at 3:08 PM, Jonathan Rajotte
<jonathan.rajotte-julien at efficios.com> wrote:
> The overwriting of LD_LIBRARY_PATH can cause failure of tests when lttng-ust is
> not installed in the default path and location is passed via LD_LIBRARY_PATH.
>
> Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien at efficios.com>
> ---
>  tests/regression/ust/ust-dl/test_ust-dl.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/regression/ust/ust-dl/test_ust-dl.py b/tests/regression/ust/ust-dl/test_ust-dl.py
> index dc73270..7117ca1 100644
> --- a/tests/regression/ust/ust-dl/test_ust-dl.py
> +++ b/tests/regression/ust/ust-dl/test_ust-dl.py
> @@ -46,8 +46,8 @@ enable_ust_tracepoint_event(session_info, "*")
>  start_session(session_info)
>
>  test_env = os.environ.copy()
> -test_env["LD_PRELOAD"] = "liblttng-ust-dl.so"
> -test_env["LD_LIBRARY_PATH"] = test_path
> +test_env["LD_PRELOAD"] += ":liblttng-ust-dl.so"
> +test_env["LD_LIBRARY_PATH"] += ":" + test_path
>  test_process = subprocess.Popen(test_path + "prog",
>                                  stdout=subprocess.PIPE, stderr=subprocess.PIPE,
>                                  env=test_env)
> --
> 2.5.0
>



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