Skip to content

Commit

Permalink
Use Windows system tar (not the Git Bash tar) (#206)
Browse files Browse the repository at this point in the history
* use windows tar, not git bash one

* correct manual patch

* rebuild

* Update src/installer.ts

Co-authored-by: Dilum Aluthge <[email protected]>

* Update installer.js

* Include the direct URL of the issue (not just the issue number)

* npm run build

---------

Co-authored-by: Dilum Aluthge <[email protected]>
Co-authored-by: Viral B. Shah <[email protected]>
  • Loading branch information
3 people authored Jun 19, 2024
1 parent 63a26ec commit ed4a842
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
4 changes: 3 additions & 1 deletion lib/installer.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 3 additions & 1 deletion src/installer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -263,7 +263,9 @@ export async function installJulia(dest: string, versionInfo, version: string, a
}
} else {
// This is the more common path. Using .tar.gz is much faster
await exec.exec('powershell', ['-Command', `tar xf ${juliaDownloadPath} --strip-components=1 -C ${dest}`])
// don't use the Git bash provided tar. Issue #205
// https://github.com/julia-actions/setup-julia/issues/205
await exec.exec('powershell', ['-Command', `& "$env:WINDIR/System32/tar" xf ${juliaDownloadPath} --strip-components=1 -C ${dest}`])
}
return dest
case 'darwin':
Expand Down

0 comments on commit ed4a842

Please sign in to comment.