Skip to content

Commit b8f3085

Browse files
committed
ci(vs-build): use actions/cache@v2.1.4 to work around the backslash problem
There is currently a problem with actions/cache@v2: when specifying a path that contains slashes (such as `compat/vcbuild/vcpkg`, which we now do in the CI/PR builds), on Windows they are converted to backslashes _and_ it uses the `tar.exe` found in the PATH. And since we install git-sdk-64-minimal, there is a GNU tar in the PATH that thinks that backslashes are a perfectly fine file name characters, not directory separators (the MSYS2 runtime will translate file name characters that are legal on Linux/Unix but illegal on Windows, mapping them into the private Unicode page). As a consequence, it fails to tar up the vcpkg files. Version 2.1.4 of actions/cache fixes these problems, by converting the backslashes back to forward slashes _and_ by insisting on using the `tar.exe` found in `C:\Windows\system32`. So: problem solved, right? Not so fast. An _unrelated_ bug was introduced in that version (actions/checkout@v2.1.4 now uses GNU tar on macOS, which seems to be unable to restore certain data shapes that were previously cached using BSD tar, see actions/cache#527 for details) and therefore v2 was rolled back to point to v2.1.3 for the moment. So let's hard-code v2.1.4 for now, hoping that the issue will be resolved soon. Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
1 parent df7d293 commit b8f3085

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/workflows/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ jobs:
146146
echo "::set-output name=id::$id"
147147
- name: Cache vcpkg
148148
id: cache-vcpkg
149-
uses: actions/cache@v2
149+
uses: actions/cache@v2.1.4
150150
with:
151151
path: compat/vcbuild/vcpkg
152152
key: vcpkg-artifacts-${{ steps.vcpkg-build-id.outputs.id }}

0 commit comments

Comments
 (0)