Skip to content

Conversation

@dzsquared
Copy link
Collaborator

A number of environment variations for Windows runners leads to fractured pathways to unzip go-sqlcmd: https://github.com/actions/toolkit/blob/1f4b3fac06b689186037c0a52e1e876e18de521d/packages/tool-cache/src/tool-cache.ts#L354

As a result, the upstream has a few open issues related to unzip on Windows:

By enforcing the zip extension for the temporary file on Windows, we're guaranteeing that the unzip option works regardless of PS and .NET versions.

Fixes #149

@dzsquared dzsquared temporarily deployed to Automation test March 10, 2023 22:04 — with GitHub Actions Inactive
@dzsquared dzsquared temporarily deployed to Automation test March 10, 2023 22:04 — with GitHub Actions Inactive
@dzsquared dzsquared temporarily deployed to Automation test March 10, 2023 22:04 — with GitHub Actions Inactive
@dzsquared dzsquared temporarily deployed to Automation test March 10, 2023 22:04 — with GitHub Actions Inactive
@dzsquared dzsquared temporarily deployed to Automation test March 10, 2023 22:04 — with GitHub Actions Inactive
@dzsquared dzsquared temporarily deployed to Automation test March 10, 2023 22:04 — with GitHub Actions Inactive
@dzsquared dzsquared merged commit 21f0f73 into master Mar 10, 2023
@dzsquared dzsquared deleted the drskwier/windows-zip-fix branch March 10, 2023 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Expand-Archive : is not a supported archive file format. .zip is the only supported archive file format.

3 participants