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

Windows virtual environment: Use SSH binaries from the Git suite #63

Merged
merged 2 commits into from
Mar 10, 2021

Conversation

mpdude
Copy link
Member

@mpdude mpdude commented Mar 4, 2021

This hopefully fixes Windows-based builds, including issue #62.

It was a much bigger rabbit hole than initially expected, and took quite some time and experimentation. It turns out there are two versions of the ssh and related binaries available in the Windows virtual environment.

C:\>C:\Windows\System32\OpenSSH\ssh.exe -v xx
OpenSSH_for_Windows_7.7p1, LibreSSL 2.6.5

C:\>C:\"Program Files"\Git\usr\bin\ssh.exe -v xx
OpenSSH_8.4p1, OpenSSL 1.1.1i  8 Dec 2020

Things seem to work when using the ssh-agent and ssh-add binaries provided as part of the Git installation, but not with the native Windows ones.

As of writing, simple git clone ... operations given in a GHA workflow file will use the ssh client from the Git suite, without further configuration. I don't know if this might change in the future; git-for-windows/git#2944 is about possibly changing this.

As a reminder to myself, just in case we need to get back to this sometime:

I don't know if "mixing" the SSH client and/or other binaries (the agent) from Windows and Git has any other side effects or implications.

🤞🏻

@leafac
Copy link

leafac commented Mar 10, 2021

@mpdude congratulations on the excellent investigative work here 👏

This was referenced Mar 10, 2021
@dokempf dokempf mentioned this pull request Jul 20, 2021
@sfullerbeckman
Copy link

I am seeing errors when trying to load multiple private keys on windows-2019. Please see these issues. #130 and actions/checkout#928

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.

3 participants