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 <[email protected]>
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 <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
---
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 '[email protected]@.git@' ${formula}
cat ${formula}
- cp ${formula} $(brew --repository homebrew/core)/Formula/
+ cp ${formula} $(brew --repository homebrew/core)/Formula/a/
done
{% endmacro %}
