You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ko...@apache.org on 2019/07/04 05:52:07 UTC

[arrow] 28/38: [Release] Set C++ libraries runtime path to LD_LIBRARY_PATH when running integration tests (#4775)

This is an automated email from the ASF dual-hosted git repository.

kou pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git

commit 14baf5e2cc43e4b32a4f0dbef8f95c4dffe255d5
Author: Wes McKinney <we...@users.noreply.github.com>
AuthorDate: Tue Jul 2 18:56:40 2019 -0500

    [Release] Set C++ libraries runtime path to LD_LIBRARY_PATH when running integration tests (#4775)
    
    This is also required (and set) when running the unit tests
    
    * Set LD_LIBRARY_PATH in integration tests
    
    * Code review [skip ci]
---
 dev/release/verify-release-candidate.sh | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/dev/release/verify-release-candidate.sh b/dev/release/verify-release-candidate.sh
index 8b25d30..0acb56e 100755
--- a/dev/release/verify-release-candidate.sh
+++ b/dev/release/verify-release-candidate.sh
@@ -486,7 +486,10 @@ test_integration() {
     INTEGRATION_TEST_ARGS=--run_flight
   fi
 
-  python integration_test.py $INTEGRATION_TEST_ARGS
+  # Flight integration test executable have runtime dependency on
+  # release/libgtest.so
+  LD_LIBRARY_PATH=$ARROW_CPP_EXE_PATH:$LD_LIBRARY_PATH \
+      python integration_test.py $INTEGRATION_TEST_ARGS
 
   popd
 }