Skip to content
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

Closed
wants to merge 3 commits into from
Closed

Conversation

eschnett
Copy link
Contributor

@eschnett eschnett commented Aug 7, 2023

Closes #73

Supersedes eschnett#1

@eschnett eschnett mentioned this pull request Aug 7, 2023
@eschnett
Copy link
Contributor Author

eschnett commented Aug 8, 2023

I checked the output of OpenSSH_jll.ssh_keygen for the new version 9, and there doesn't seem to be any difference to previous versions.

@eschnett
Copy link
Contributor Author

eschnett commented Aug 9, 2023

@mortenpi Can you approve the CI workflow?

@mortenpi
Copy link
Member

LGTM. Could you also bump the patch version and add a brief note to CHANGELOG.md? We can then tag a new version right away.

@eschnett
Copy link
Contributor Author

@mortenpi Done. Can you approve the new CI workflow?

CHANGELOG.md Show resolved Hide resolved
@mortenpi mortenpi enabled auto-merge (squash) August 14, 2023 05:07
@eschnett
Copy link
Contributor Author

@mortenpi Can you approve the CI workflow?

@mortenpi
Copy link
Member

mortenpi commented Aug 14, 2023

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.

@eschnett
Copy link
Contributor Author

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 stdout or stderr, I conjecture that ssh-keygen.exe did not actually start. This could point to a problem with shared libraries.

Maybe this JuliaPackaging/Yggdrasil#7204 will correct the problem.

@giordano
Copy link
Contributor

This is difficult to debug for me since I don't have access to a Windows machine.

Usual reminder that https://github.com/mxschmitt/action-tmate is a thing.

For the record, the problem may be this missing libcrypto.dll

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

@giordano
Copy link
Contributor

JuliaPackaging/Yggdrasil#9482 should fix the issue

@giordano giordano mentioned this pull request Sep 23, 2024
@giordano
Copy link
Contributor

I opened #91 to resolve the merge conflicts.

auto-merge was automatically disabled September 23, 2024 23:22

Pull request was closed

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.

Allow OpenSSH @9
3 participants