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


##########
dev/tasks/r/github.packages.yml:
##########
@@ -280,10 +319,33 @@ jobs:
         with:
           install-r: false
       {{ macros.github_setup_local_r_repo(false, false)|indent }}
+      - name: Get sccache
+        shell: bash
+        run: |
+          SCCACHE_BUILD=unknown-linux-musl
+          
SCCACHE_URL="https://github.com/mozilla/sccache/releases/download/v0.3.0/sccache-v0.3.0-x86_64-$SCCACHE_BUILD.tar.gz";
+
+          # Download archive and checksum
+          curl -L $SCCACHE_URL --output sccache.tar.gz
+          curl -L $SCCACHE_URL.sha256 --output sccache.tar.gz.sha256
+
+          SCCACHE_ARCHIVE=sccache.tar.gz
+          echo "$(cat $SCCACHE_ARCHIVE.sha256) $SCCACHE_ARCHIVE" | sha256sum 
--check --status
+
+          mkdir -p sccache
+          tar -xzvf $SCCACHE_ARCHIVE --strip-component=1 --directory sccache
+
+          echo "$(pwd)/sccache" >> $GITHUB_PATH
+          echo "SCCACHE_PATH=$(pwd)/sccache/sccache.exe" >> $GITHUB_ENV
+      - run: sudo apt update && sudo apt install libcurl4-openssl-dev

Review Comment:
   why is this now required? Should be done on the previous step (install 
sscache)?



##########
cpp/cmake_modules/DefineOptions.cmake:
##########
@@ -111,7 +111,10 @@ if(ARROW_DEFINE_OPTIONS)
 
   define_option(ARROW_POSITION_INDEPENDENT_CODE
                 "Whether to create position-independent target" ON)
-
+  
+  define_option(ARROW_USE_SCCACHE "Use sccache when compiling (if available), 
takes precdent\

Review Comment:
   ```suggestion
     define_option(ARROW_USE_SCCACHE "Use sccache when compiling (if 
available), takes precedent\
   ```



##########
dev/tasks/r/github.packages.yml:
##########
@@ -155,11 +186,19 @@ jobs:
           rtools-version: {{ '${{ matrix.r_version.rtools }}' }}
           Ncpus: 2
       {{ macros.github_setup_local_r_repo(false, true)|indent }}
+      - name: Install sccache
+        if: startsWith(matrix.platform, 'macos')
+        run: brew install sccache

Review Comment:
   could we install on windows following the same approach we don on line 132: 
`- name: Get sccache`?
   Maybe we could create our own action for that. I am just thinking out loud, 
not required for this PR.



-- 
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-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to