Skip to content

Commit

Permalink
Don't add rtools in PATH as this could mess other actions like Julia
Browse files Browse the repository at this point in the history
julia-actions/setup-julia@v1.9.5 is using `tar` for Windows installation
now but uses any `tar` from PATH. However, it requires a `tar.exe` that
can handle Windows paths with drive letters.

In our workflow, `tar` is found in `/usr/bin/tar` from rtools being
added to PATH. This leads to error as it does not support the drive
letters.

Not adding rtools to PATH will solve, and should not impact the workflow
as pak is used for R packages installation.

More details about this issue at julia-actions/setup-julia#205
  • Loading branch information
cderv committed Feb 1, 2024
1 parent ec53736 commit 8cad966
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion .github/workflows/test-smokes.yml
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,8 @@ jobs:
with:
r-version: "4.3.2"
use-public-rspm: true
# required to avoid rtools bin in path
windows-path-include-rtools: false

- name: Install node (for Playwright, MECA)
uses: actions/setup-node@v3
Expand Down Expand Up @@ -153,7 +155,7 @@ jobs:
uses: browser-actions/setup-chrome@latest

- name: Setup Julia
uses: cderv/setup-julia@fix-tar
uses: julia-actions/setup-julia@v1
with:
version: "1.10"

Expand Down

0 comments on commit 8cad966

Please sign in to comment.