[PATCH] test/symbol-hiding: prepend new directory to LD_LIBRARY_PATH

[PATCH] test/symbol-hiding: prepend new directory to LD_LIBRARY_PATH

[Tomi Ollila]

Tester may have set LD_LIBRARY_PATH to find libraries required
by notmuch. Therefore add $TEST_DIRECTORY/../lib to the beginning
of current list of library paths in $LD_LIBRARY_PATH before running
symbol-test.
---
 test/symbol-hiding |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/test/symbol-hiding b/test/symbol-hiding
index 68f0d1b..7fa7b2a 100755
--- a/test/symbol-hiding
+++ b/test/symbol-hiding
@@ -12,7 +12,7 @@ test_description='exception symbol hiding'
 . ./test-lib.sh
 
 run_test(){
-    result=$(LD_LIBRARY_PATH=$TEST_DIRECTORY/../lib $TEST_DIRECTORY/symbol-test 2>&1)
+    result=$(LD_LIBRARY_PATH="$TEST_DIRECTORY/../lib${LD_LIBRARY_PATH:+:$LD_LIBRARY_PATH}" $TEST_DIRECTORY/symbol-test 2>&1)
 }
 
 output="A Xapian exception occurred opening database: Couldn't stat 'fakedb/.notmuch/xapian'
-- 
1.7.7.4

Re: [PATCH] test/symbol-hiding: prepend new directory to LD_LIBRARY_PATH

[David Bremner]

On Fri, 30 Dec 2011 18:12:45 +0200, Tomi Ollila <tomi.ollila@iki.fi> wrote:
> Tester may have set LD_LIBRARY_PATH to find libraries required
> by notmuch. Therefore add $TEST_DIRECTORY/../lib to the beginning
> of current list of library paths in $LD_LIBRARY_PATH before running
> symbol-test.

Hmm. People should just install things properly ;).

pushed.

d