kou commented on code in PR #868:
URL: https://github.com/apache/arrow-java/pull/868#discussion_r2400717270
##########
.github/workflows/rc.yml:
##########
@@ -168,7 +168,7 @@ jobs:
fail-fast: false
matrix:
platform:
- - { runs_on: macos-13, arch: "x86_64"}
+ - { runs_on: macos-latest, arch: "aarch_64"}
Review Comment:
We can remove this pattern entirely.
We just drop support for Intel macOS.
##########
.github/workflows/rc.yml:
##########
@@ -233,10 +229,10 @@ jobs:
# tries to replace /usr/local/bin/2to3 and so on and causes
# a conflict error.
brew update
- for python_package in $(brew list | grep python@ | sort -r); do
+ for python_package in $(brew list | grep python@); do
brew install --overwrite ${python_package}
done
- brew install --overwrite python3
+ brew install --overwrite python
Review Comment:
We can remove this.
##########
.github/workflows/rc.yml:
##########
Review Comment:
We may be able to remove this entirely now.
##########
.github/workflows/rc.yml:
##########
@@ -233,10 +229,10 @@ jobs:
# tries to replace /usr/local/bin/2to3 and so on and causes
# a conflict error.
brew update
- for python_package in $(brew list | grep python@ | sort -r); do
+ for python_package in $(brew list | grep python@); do
brew install --overwrite ${python_package}
done
- brew install --overwrite python3
+ brew install --overwrite python
if [ "$(uname -m)" = "arm64" ]; then
Review Comment:
We can remove this `if`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
