actions
/
toolkit
mirror of https://github.com/actions/toolkit
1
0
Fork 0
Code

mv rather than cp on non-windows

pull/1504/head
Ian Butterworth 2023-08-23 11:58:44 -04:00
parent 7b617c260d
commit 8c202adf94
1 changed files with 20 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
packages/tool-cache/src/tool-cache.ts
View File

@ -431,12 +431,19 @@ export async function cacheDir(
// Create the tool dir
const destPath: string = await _createToolPath(tool, version, arch)
if (IS_WINDOWS) {
// copy each child item. do not move. move can fail on Windows
// due to anti-virus software having an open handle on a file.
for (const itemName of fs.readdirSync(sourceDir)) {
const s = path.join(sourceDir, itemName)
await io.cp(s, destPath, {recursive: true})
}
} else {
for (const itemName of fs.readdirSync(sourceDir)) {
const s = path.join(sourceDir, itemName)
await io.mv(s, destPath, {recursive: true})
}
}
// write .complete
_completeToolPath(tool, version, arch)
@ -473,11 +480,16 @@ export async function cacheFile(
// create the tool dir
const destFolder: string = await _createToolPath(tool, version, arch)
// copy instead of move. move can fail on Windows due to
// anti-virus software having an open handle on a file.
const destPath: string = path.join(destFolder, targetFile)
core.debug(`destination file ${destPath}`)
if (IS_WINDOWS) {
// copy instead of move. move can fail on Windows due to
// anti-virus software having an open handle on a file.
await io.cp(sourceFile, destPath)
} else {
await io.mv(sourceFile, destPath)
}
// write .complete
_completeToolPath(tool, version, arch)