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 2026/10/06 05:17:58 UTC
(arrow) branch main updated: GH-51751: [CI][Packaging] Fix formula path for homebrew-cpp job (#51756)
This is an automated email from the ASF dual-hosted git repository.
kou pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/main by this push:
new 0e9fca31ba3 GH-51751: [CI][Packaging] Fix formula path for homebrew-cpp job (#51756)
0e9fca31ba3 is described below
commit 0e9fca31ba3081bf198ba79f9e3c985545154a5b
Author: tadeja <ta...@users.noreply.github.com>
AuthorDate: Tue Oct 6 07:17:50 2026 +0200
GH-51751: [CI][Packaging] Fix formula path for homebrew-cpp job (#51756)
### Rationale for this change
Fix #51751
homebrew-core's formulae got letter-folders in 2023 but our macro from commit 5ad5ddc kept copying to top-level `Formula/` hence homebrew-cpp job tested homebrew-core's formulae instead of both apache-arrow.rb and apache-arrow-glib.rb in `dev/tasks/homebrew-formulae/`
### What changes are included in this PR?
Copy to `Formula/a/` and
change to `formula_opt_prefix("llvm")` per audit check
### Are these changes tested?
Yes.
### Are there any user-facing changes?
No.
### Was AI used for this PR?
**PR code and description written by:**
- [X] Human
- [ ] AI
**Reviewed before submission by:**
- [X] Human
- [x] AI
- [ ] Not reviewed
* GitHub Issue: #51751
Authored-by: Tadeja Kadunc <ta...@gmail.com>
Signed-off-by: Sutou Kouhei <ko...@clear-code.com>
---
dev/tasks/homebrew-formulae/apache-arrow.rb | 2 +-
dev/tasks/macros.jinja | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/dev/tasks/homebrew-formulae/apache-arrow.rb b/dev/tasks/homebrew-formulae/apache-arrow.rb
index d2b99fd4757..544a5e318d8 100644
--- a/dev/tasks/homebrew-formulae/apache-arrow.rb
+++ b/dev/tasks/homebrew-formulae/apache-arrow.rb
@@ -71,7 +71,7 @@ class ApacheArrow < Formula
# We set `ARROW_ORC=OFF` because it fails to build with Protobuf 27.0
args = %W[
-DCMAKE_INSTALL_RPATH=#{rpath}
- -DLLVM_ROOT=#{Formula["llvm"].opt_prefix}
+ -DLLVM_ROOT=#{formula_opt_prefix("llvm")}
-DARROW_DEPENDENCY_SOURCE=SYSTEM
-DARROW_ACERO=ON
-DARROW_COMPUTE=ON
diff --git a/dev/tasks/macros.jinja b/dev/tasks/macros.jinja
index 5b74999d958..161c3eeb9ad 100644
--- a/dev/tasks/macros.jinja
+++ b/dev/tasks/macros.jinja
@@ -175,7 +175,7 @@ env:
# Make sure there's only one
sed -i '' -E -e 's@.git.git@.git@' ${formula}
cat ${formula}
- cp ${formula} $(brew --repository homebrew/core)/Formula/
+ cp ${formula} $(brew --repository homebrew/core)/Formula/a/
done
{% endmacro %}