-
Notifications
You must be signed in to change notification settings - Fork 56
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
ssh-keygen relies on a binary #147
Comments
This is a bit of a duplication of one of the checkboxes in #64. |
Oh interesting. I suppose splitting it out gives it more visibility as an easy task. |
+1 |
Anyone with windows that could test this patch? 👷 |
Update: @LinusU's patch works but has been blocked by this issue: nodejs/node-v0.x-archive#4051 Since Windows exporting OpenSSL labels is unlikely to happen, and this is a minor system component, I think the addition of more binary dependencies isn't necessary. Instead, I'm proposing we merge #429 which adds support for a pure-JS keygen. |
Yep, we're all good! |
Turns out it was really hard to find a
ssh-keygen
binary on Windows. It's in Git Bash, but not in chocolately or made available to Powershell or Bash.The text was updated successfully, but these errors were encountered: