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 373ce81bd4 MINOR: [Java][CI] Update the order of commands in Java-Jars
CI (#43462)
373ce81bd4 is described below
commit 373ce81bd40ee9bfd07b8cbec9821526096296bc
Author: Vibhatha Lakmal Abeykoon <[email protected]>
AuthorDate: Mon Jul 29 10:31:11 2024 +0530
MINOR: [Java][CI] Update the order of commands in Java-Jars CI (#43462)
### Rationale for this change
`googletest` is installed by the `Brewfile` associated with cpp not Java.
So move the order of uninstalling in that order.
### What changes are included in this PR?
Change the order of commands to suit the installation objectives.
### Are these changes tested?
Existing tests.
### Are there any user-facing changes?
No
Authored-by: Vibhatha Lakmal Abeykoon <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
---
dev/tasks/java-jars/github.yml | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/dev/tasks/java-jars/github.yml b/dev/tasks/java-jars/github.yml
index ba988f8931..3747788c09 100644
--- a/dev/tasks/java-jars/github.yml
+++ b/dev/tasks/java-jars/github.yml
@@ -138,14 +138,13 @@ jobs:
# used on test We uninstall Homebrew's Protobuf to ensure using
# bundled Protobuf.
brew uninstall protobuf
-
- brew bundle --file=arrow/java/Brewfile
-
# We want to use the bundled googletest for static linking. Since
# both BUNDLED and brew options are enabled, it could cause a
conflict
# when there is a version mismatch.
# We uninstall googletest to ensure using the bundled googletest.
brew uninstall googletest
+
+ brew bundle --file=arrow/java/Brewfile
- name: Build C++ libraries
env:
{{ macros.github_set_sccache_envvars()|indent(8) }}
