You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "assignUser (via GitHub)" <gi...@apache.org> on 2023/07/31 17:57:03 UTC
[GitHub] [arrow] assignUser commented on a diff in pull request #36693: GH-36692: [CI][Packaging] Pin gemfury to 0.12.0 due to issue with faraday dependency
assignUser commented on code in PR #36693:
URL: https://github.com/apache/arrow/pull/36693#discussion_r1279689672
##########
dev/tasks/macros.jinja:
##########
@@ -137,11 +137,25 @@ on:
run: |
sudo apt update
sudo apt install -y ruby-full
+ - name: Set up Ruby by GitHub Actions
+ if: runner.arch == 'X64' && runner.os != 'macOS'
+ uses: ruby/setup-ruby@v1
+ with:
+ ruby-version: "ruby"
+ - name: Install gemfury client on ARM self-hosted
+ if: runner.arch != 'X64'
+ run: |
+ # GH-36692: Pin gemfury due to wrong faraday dependency declaration.
+ gem install --user-install gemfury -v 0.12.0
Review Comment:
Is there a downside to the user install option? If not should we just install that way on all runners?
Alternatively these steps could be merged and the option inserted like this `{{ "${{ runner.arch == 'X64' && '--user-install' || ''}}" }}`
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org