assignUser commented on code in PR #13379:
URL: https://github.com/apache/arrow/pull/13379#discussion_r898200657
##########
.github/workflows/r.yml:
##########
@@ -292,75 +280,23 @@ jobs:
cd r/windows
ls *.zip | xargs -n 1 unzip -uo
rm -rf *.zip
- - name: Setup ccache
- shell: bash
- run: |
- ci/scripts/ccache_setup.sh
- echo "CCACHE_DIR=$(cygpath --absolute --windows ccache)" >>
$GITHUB_ENV
- # We must enable actions/cache before r-lib/actions/setup-r to ensure
- # using system tar instead of tar provided by Rtools.
- # We can use tar provided by Rtools when we drop support for Rtools 3.5.
- # Because Rtools 4.0 or later has zstd. actions/cache requires zstd
- # when tar is GNU tar.
- - name: Cache ccache
- uses: actions/cache@v2
- with:
- path: ccache
- key: r-${{ matrix.config.rtools }}-ccache-mingw-${{
hashFiles('cpp/**') }}
- restore-keys: r-${{ matrix.config.rtools }}-ccache-mingw-
- uses: r-lib/actions/setup-r@v2
with:
r-version: ${{ matrix.config.rversion }}
rtools-version: ${{ matrix.config.rtools }}
+ use-public-rspm: true
Ncpus: 2
- - name: Make R tests verbose
- # If you get a segfault/mysterious test Execution halted,
- # make this `true` to see where it dies.
- if: false
- shell: cmd
- run: |
- cd r/tests
- sed -i.bak -E -e 's/"arrow"/"arrow", reporter = "location"/'
testthat.R
- rm -f testthat.R.bak
- - name: Install cpp11 (on R 3.6)
- # Since we force installation of binary packages below, dependency
versions
- # are frozen for old versions of R. We need newer cpp11 than is
available as
- # "binary" (though it doesn't matter because the cpp11 R package is just
a
- # vehicle for the header-only C++ code.
- if: ${{ matrix.config.rtools == 35 }}
- shell: Rscript {0}
- run: install.packages("cpp11", type = "source")
- - name: Prune dependencies (on R 3.6)
- if: ${{ matrix.config.rtools == 35 }}
- shell: Rscript {0}
- run: |
- # To prevent the build from timing out, let's prune some optional
deps (and their possible version requirements)
- setwd("r")
- # create a backup to use later
- file.copy("DESCRIPTION", "DESCRIPTION.bak")
- d <- read.dcf("DESCRIPTION")
- to_prune <- c("duckdb", "DBI", "dbplyr", "decor", "knitr",
"rmarkdown", "pkgload", "reticulate")
- pattern <- paste0("\\n?", to_prune, "( \\([^,]*\\))?,?", collapse
= "|")
- d[,"Suggests"] <- gsub(pattern, "", d[,"Suggests"])
- write.dcf(d, "DESCRIPTION")
- - name: R package cache
- uses: actions/cache@v2
+ - uses: r-lib/actions/setup-r-dependencies@v2
+ env:
+ GITHUB_PAT: "${{ github.token }}"
with:
- path: |
- ${{ env.R_LIBS_USER }}/*
- key: r-${{ matrix.config.rtools }}-R-LIBS-${{
hashFiles('r/DESCRIPTION') }}
- restore-keys: r-${{ matrix.config.rtools }}-R-LIBS-
- - name: Install R package dependencies
- shell: Rscript {0}
- run: |
- # options(pkgType="win.binary") # processx doesn't have a binary for
UCRT yet
- install.packages(c("remotes", "rcmdcheck"))
- remotes::install_deps("r", dependencies = TRUE)
- - name: Restore DESCRIPTION for 3.6
- if: ${{ matrix.config.rtools == 35 }}
- run: |
- rm r/DESCRIPTION
- mv r/DESCRIPTION.bak r/DESCRIPTION
+ # For some arcane reason caching does not work on the windows runners
+ # most likely due to https://github.com/actions/cache/issues/815
+ cache: false
+ working-directory: 'r'
+ extra-packages: |
+ any::rcmdcheck
+ github::r-lib/[email protected]
Review Comment:
Looking at the original problem I don't think this is necessary, pak should
install the newest version from src on 3.6, as we no longer force binary
install. I'll test.
--
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]
