[GitHub] [arrow] raulcd commented on a diff in pull request #36693: GH-36692: [CI][Packaging] Pin gemfury to 0.12.0 due to issue with faraday dependency

["raulcd (via GitHub)" <gi...@apache.org>]

raulcd commented on code in PR #36693:
URL: https://github.com/apache/arrow/pull/36693#discussion_r1280291295


##########
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:
   If we want to use the `setup-ruby` action, to ensure recent ruby versions, the step fails when trying to use `--user-install` that's why we are separating them. As suggested by Kou I've also added adding the user install dir to the path on the ARM step.
   I also prefer to be more explicit on the difference between ARM and non-ARM on the different step. I feel is more readable than:
   `{{ "${{ 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