-
-
Notifications
You must be signed in to change notification settings - Fork 283
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
"tar (child): Cannot connect to d: resolve failed" error when intalling on Windows #28
Comments
One big difference between those two runs:
and
So We're using the official Line 38 in f010ae4
So it must be a bug of that or of the GitHub Actions image. |
Seems very similar to actions/toolkit#194 |
Been there, done that. There's more issues with the 'git' tar also. I think it won't do lzma? Was just going to do the following around ruby_builder.js:38: const downloadPath = await tc.downloadTool(url)
if (os.platform() === 'win32') {
let args = [ '-xz', '-C', rubiesDir, '-f', downloadPath ]
await exec.exec('C:\\Windows\\system32\\tar.exe', args)
} else {
await tc.extractTar(downloadPath, rubiesDir)
} |
@MSP-Greg That looks good, could you make a PR? |
I can reproduce: https://github.com/ruby/setup-ruby/runs/475090092?check_suite_focus=true I guess they changed the PATH on Windows images. |
Yes. As mentioned, the git tar is messy, I've had to feed it 'nix' style paths. As I've dug more into some of the custom actions, some seem to warrant replacing. For instance, one had a dependency on another one, and the dependency had been updated a couple of times, but it wasn't accounted for in the main action. IOW, there's often unneeded layers added. Like with this, I wasn't sure if node had a library like Ruby has, or whether they were just shelling out. We might as well shell out and use the native Windows tar... |
I filed an issue, it's a bug of the new GitHub Actions Windows images: actions/runner-images#480 |
See PR #29 |
Fixed by #29, released in https://github.com/ruby/setup-ruby/releases/tag/v1.20.1, v1 branch updated, so this should work fine now. |
Thanks for all these investigations & a quick fix! |
Failure log: https://github.com/slonopotamus/asciidoctor-epub3/runs/474587790?check_suite_focus=true#step:3:6
No changes were done in repo since previous run 3 days ago that worked successfully: https://github.com/slonopotamus/asciidoctor-epub3/runs/467944869?check_suite_focus=true#step:3:5
There's also evidence that it worked yesterday: https://github.com/asciidoctor/asciidoctor-pdf/runs/471779090
Same failure in a different project: https://github.com/asciidoctor/asciidoctor-pdf/runs/474207029#step:3:8
The text was updated successfully, but these errors were encountered: