FIX: Extract zip win #827
FIX: Extract zip win #827
Conversation
|
@priyagupta108 I took the liberty to take a look at #819 and I think this changes should correct the issue. I tested it on my repo and it works fine (didn't do intensive testing though). |
There was a problem hiding this comment.
This will solve so many issues! Windows self-hosted runners could no longer use "v5" if the Python version requested was not available in the cache or in the system already. We can finally use the action as is without having to stick to "v4" or implement workarounds!
Related to actions#819 Changes consist in reordering extract utils to first attemps ExtractToDirectory and then Expand-Archive. For some reason the download CPython release was not correctly handled by Expand-Archive. This could be happening if the zip file was compressed using an unsupported compression method (e.g. through `zip` with Unix/linux).
SMoraisAnsys
force-pushed
the
fix/windows_extract_cpython_release
branch
from
606bed9
to
4de2ef7
Compare
|
I took the liberty to reproduce the error (fixed by this PR) on a github runner. The associated branch (https://github.com/SMoraisAnsys/setup-python/tree/test/no-pwshpath) consists in removing a part of the code of PR that can be used to oversee the changes : SMoraisAnsys#1 |
|
@actions/setup-actions-team could we get this reviewed and merged soon? |
Description:
Changes consist in reordering extract utils to first attemps ExtractToDirectory and then Expand-Archive. For some reason the download CPython release was not correctly handled by Expand-Archive. This could be happening if the zip file was compressed using an unsupported compression method (e.g. through
zipwith Unix/Linux).Related issue: #819
Check list: