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