[ 
https://issues.apache.org/jira/browse/ARROW-9572?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17167587#comment-17167587
 ] 

Kouhei Sutou commented on ARROW-9572:
-------------------------------------

How about forcing using the system libc++ like v8 does?
https://github.com/Homebrew/homebrew-core/blob/master/Formula/v8.rb#L101-L102

{noformat}
diff --git a/dev/tasks/homebrew-formulae/apache-arrow.rb 
b/dev/tasks/homebrew-formulae/apache-arrow.rb
index c38b8ce51..f078e49d4 100644
--- a/dev/tasks/homebrew-formulae/apache-arrow.rb
+++ b/dev/tasks/homebrew-formulae/apache-arrow.rb
@@ -24,6 +24,8 @@ class ApacheArrow < Formula
 
   def install
     ENV.cxx11
+    # link against system libc++ instead of llvm provided libc++
+    ENV.remove "HOMEBREW_LIBRARY_PATHS", Formula["llvm"].opt_lib
     args = %W[
       -DARROW_FLIGHT=ON
       -DARROW_GANDIVA=ON
{noformat}

For testing, it seems that we can use {{brew test-bot --only-formulae}} for 
checking correct build dependency.

> [CI][Homebrew] Properly enable Gandiva and improve testing
> ----------------------------------------------------------
>
>                 Key: ARROW-9572
>                 URL: https://issues.apache.org/jira/browse/ARROW-9572
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: C++ - Gandiva, Packaging
>            Reporter: Neal Richardson
>            Priority: Major
>             Fix For: 2.0.0
>
>
> ARROW-9086 enabled Gandiva in our Homebrew formula, but when I tried to add 
> that to the official Homebrew formula at release time, it failed. See some 
> discussion at https://github.com/Homebrew/homebrew-core/pull/58581, though 
> unfortunately the build logs are gone. 
> It turns out that the testing that Homebrew does is more thorough than the 
> install/audit we do in CI. See 
> https://github.com/Homebrew/homebrew-core/pull/58581/checks?check_run_id=915732878
>  for example. They install, build the bottle, then remove all dependencies 
> and reinstall the bottle. Since this failed, what I think it means is that 
> `llvm` is not a build-only dependency for Gandiva--it built but couldn't run 
> successfully because `llvm` had been removed.
> cc [~kou]



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to