You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ks...@apache.org on 2019/02/17 12:37:11 UTC

[arrow] branch master updated: ARROW-4598: [CI] Remove needless LLVM_DIR for macOS

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

kszucs 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 a7e93a5  ARROW-4598: [CI] Remove needless LLVM_DIR for macOS
a7e93a5 is described below

commit a7e93a5f25c609179ff849315ba18486b6a4fae9
Author: Kouhei Sutou <ko...@clear-code.com>
AuthorDate: Sun Feb 17 13:36:49 2019 +0100

    ARROW-4598: [CI] Remove needless LLVM_DIR for macOS
    
    We can detect LLVM installed by Homebrew automatically in CMake.
    
    Author: Kouhei Sutou <ko...@clear-code.com>
    
    Closes #3674 from kou/travis-remove-needless-llvm-dir and squashes the following commits:
    
    70d4bf67 <Kouhei Sutou>  Remove needless LLVM_DIR for macOS
---
 ci/travis_before_script_cpp.sh | 1 -
 1 file changed, 1 deletion(-)

diff --git a/ci/travis_before_script_cpp.sh b/ci/travis_before_script_cpp.sh
index a1e407f..c7c3912 100755
--- a/ci/travis_before_script_cpp.sh
+++ b/ci/travis_before_script_cpp.sh
@@ -151,7 +151,6 @@ else
     if [ "$using_homebrew" = "yes" ]; then
 	# build against homebrew's boost if we're using it
 	export BOOST_ROOT=$(brew --prefix boost)
-	export LLVM_DIR=$(brew --prefix llvm@6)/lib/cmake/llvm
 	export THRIFT_HOME=$(brew --prefix thrift)
     fi
     cmake $CMAKE_COMMON_FLAGS \