Skip to content

MINOR: Don't install Homebrew's Node.js for JNI macOS build - #1270

Merged
kou merged 1 commit into
apache:mainfrom
jbonofre:jni-macos-skip-node
Aug 31, 2026
Merged

MINOR: Don't install Homebrew's Node.js for JNI macOS build#1270
kou merged 1 commit into
apache:mainfrom
jbonofre:jni-macos-skip-node

Conversation

@jbonofre

Copy link
Copy Markdown
Member

The JNI macos-15-intel x86_64 job currently fails on main and on every open PR while installing dependencies:

==> Installing node
Error: node: no bottle available!
This is a Tier 3 configuration:
https://docs.brew.sh/Support-Tiers#tier-3
Installing node has failed!
`brew bundle` failed! 1 Brewfile dependency failed to install

apache/arrow's cpp/Brewfile lists brew "node", and Homebrew treats x86_64 macOS as a tier 3 configuration and no longer builds a bottle for it. brew bundle doesn't build from source, so the step aborts before anything is built.

We don't need Node.js for the JNI build, so this skips installing it via HOMEBREW_BUNDLE_BREW_SKIP, the same way #1269 did for aws-sdk-cpp and gRPC.

Failing runs for reference:

Homebrew doesn't provide a bottle for node on x86_64 macOS because
Homebrew treats x86_64 macOS as a tier 3 configuration. So "brew
bundle --file=arrow/cpp/Brewfile" fails with "node: no bottle
available!" and the "JNI macos-15-intel x86_64" job can't install
dependencies.
We don't need Node.js for the JNI build. So we skip installing it like
we already do for aws-sdk-cpp and gRPC.
@jbonofrejbonofre added the bug-fix PRs that fix a big. label Aug 30, 2026
@apacheapache deleted a comment from github-actionsBotAug 30, 2026
@github-actionsgithub-actionsBot added this to the 20.0.0 milestone Aug 30, 2026
@jbonofre

Copy link
Copy Markdown
MemberAuthor

@lidavidm@kou same story as the other PR 😄

kou
kou approved these changes Aug 31, 2026

@koukou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@kou
kou merged commit 4e8faf7 into apache:mainAug 31, 2026
23 of 25 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fixPRs that fix a big.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@jbonofre@kou@lidavidm