You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by we...@apache.org on 2019/07/02 23:56:46 UTC

[arrow] branch master updated: [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.

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


The following commit(s) were added to refs/heads/master by this push:
     new c645a37  [Release] Set C++ libraries runtime path to LD_LIBRARY_PATH when running integration tests (#4775)
c645a37 is described below

commit c645a3791448a2498c4b5e6acd6ff70ea493c8fd
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
 }