Skip to content

Commit

Permalink
Fix saving package cache on Windows & older R (#944)
Browse files Browse the repository at this point in the history
* Empty commit to trigger a GHA run

* Add interactive debugging

Temporarily for now.

* setup-r-dependencies: set path to avoid Rtools tar & zstd

* setup-r-dependencies: do not prioritize c:/windows/system32 path

Makes setup-quarto fail (?!).

* setup-r-dependencies: add back the tar path change for R 3.x

Probably still needed.

* Remove debug-shell

* Update NEWS
  • Loading branch information
gaborcsardi authored Nov 9, 2024
1 parent 9073210 commit e9f20ef
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
3 changes: 3 additions & 0 deletions NEWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,9 @@
* `[setup-renv]` now correctly accepts `bypass-cache: never`, as stated
in the documentation.

* `[setup-r-dependencies]` now saves the package cache correctly on
Windows with older R (#940).

# `v2.10.1` (2024-08-08)

* `[setup-r-dependencies]` now pins `quarto-dev/quarto-actions/setup`
Expand Down
6 changes: 6 additions & 0 deletions setup-r-dependencies/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -174,6 +174,12 @@ runs:
shell: Rscript {0}
working-directory: ${{ inputs.working-directory }}

- name: Adjust PATH to avoid using zstd from Rtools
if: runner.os == 'Windows'
shell: bash
run: |
echo 'C:/tools/zstd' >> $GITHUB_PATH
- name: R package cache
if: inputs.cache == 'true'
uses: actions/cache@v4
Expand Down

0 comments on commit e9f20ef

Please sign in to comment.