-
Notifications
You must be signed in to change notification settings - Fork 20
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
Allow OpenSSH @9 #74
Allow OpenSSH @9 #74
Conversation
I checked the output of |
@mortenpi Can you approve the CI workflow? |
LGTM. Could you also bump the patch version and add a brief note to |
@mortenpi Done. Can you approve the new CI workflow? |
@mortenpi Can you approve the CI workflow? |
It looks like OpenSSH v9 doesn't work on Windows? If we'd get the Windows CI to pass, then this could be merged and tagged as far as I'm concerned. |
This is difficult to debug for me since I don't have access to a Windows machine. Given that there is no error message on Maybe this JuliaPackaging/Yggdrasil#7204 will correct the problem. |
Usual reminder that https://github.com/mxschmitt/action-tmate is a thing. For the record, the problem may be this missing julia> using DependencyWalker
julia> Library(OpenSSH_jll.ssh_keygen_path)
◼ C:\Users\runneradmin\.julia\artifacts\bed08e38e883f52775d130e1f289ee06485dbc6c\bin\ssh-keygen.exe
◼ C:\Windows\System32\CRYPT32.dll
✗ libcrypto.dll (NOT FOUND)
◼ C:\Windows\System32\USER32.dll
◼ C:\Windows\System32\GDI32.dll
◼ C:\Windows\System32\KERNELBASE.dll
◼ C:\Windows\System32\WS2_32.dll
◼ C:\Windows\System32\RPCRT4.dll
◼ C:\Windows\System32\KERNELBASE.dll
◼ C:\Windows\System32\KERNEL32.DLL
◼ C:\Windows\System32\KERNELBASE.dll
◼ C:\Windows\System32\ADVAPI32.dll
◼ C:\Windows\System32\KERNEL32.DLL
◼ C:\Windows\System32\KERNELBASE.dll
◼ C:\Windows\System32\RPCRT4.dll
◼ C:\Windows\System32\KERNELBASE.dll
◼ C:\Windows\System32\msvcrt.dll
◼ C:\Windows\System32\KERNELBASE.dll
◼ C:\Windows\System32\KERNELBASE.dll
◼ C:\Windows\System32\sechost.dll
◼ C:\Windows\System32\bcrypt.dll |
JuliaPackaging/Yggdrasil#9482 should fix the issue |
I opened #91 to resolve the merge conflicts. |
Pull request was closed
Closes #73
Supersedes eschnett#1