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.

Sign up for GitHub

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

Jump to bottom

SSH_AGENT_PID does not correspond to process Id #21

Open
technic opened this issue Sep 14, 2019 · 3 comments
Open

SSH_AGENT_PID does not correspond to process Id #21

technic opened this issue Sep 14, 2019 · 3 comments

Comments

@technic
Copy link

technic commented Sep 14, 2019 •
edited
Loading

System Details

  • posh-sshell version/path: 0.3.1 ~\Documents\WindowsPowerShell\Modules\posh-sshell\0.3.1
  • PowerShell version: 5.1.14393.3053
  • OS: Microsoft Windows NT 10.0.14393.0
  • OpenSSH / Putty version: git version 2.23.0.windows.1

Issue Description

I have ssh-agent from git for windows. Since version 2.22 SSH_AGENT_PID does not correspond to process Id (git-for-windows/git#2274). Other method to get process Id is required.
@JeremySkinner
Copy link
Collaborator

I don't personally have time to work on this at the moment (I use Linux at my current job so not using git for windows anymore), but will happily accept a pull request if you (or someone else) wants to take a look at it.

@technic
Copy link
Author

technic commented Sep 16, 2019

If only I knew how to fix it.

@JeremySkinner
Copy link
Collaborator

To be honest these days you’re probably better off using the OpenSSH agent that ships with Windows 10 ad this automatically handles starting the agent for you as a windows service. Much easier to work with than the one that comes with git for windows as you don’t have to deal with any auto start scripts.

technic added a commit to technic/posh-git that referenced this issue Sep 16, 2019
@technic
Make SSH_AGENT_PID mapping to Windows PID
f9556db 
Fixes dahlbyk/posh-sshell#21
PID and WINPID mismatch was introduced in the recent cygwin release,
see git-for-windows/git#2274
@technic technic mentioned this issue Sep 16, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@JeremySkinner @technic