Skip to content
This repository was archived by the owner on Nov 9, 2017. It is now read-only.

Conversation

@mattyclarkson
Copy link
Contributor

I have written a batch file that starts the ssh-agent and authenticates keys on the system with the ssh-agent in the MS-DOS prompt. I have put the script in the cmd folder as that is the one that is added to the native prompt. If the script is double-clicked on in explorer it starts a new prompt else if invoked from a current prompt it will just authenticate the keys.

It looks up the ssh-agent in the tasklist to make sure that multiple agents are not started.

It's not perfect, but I wondered if it would be useful to the project. Happy to change it in any way you would like if you would find it a useful addition.

@mattyclarkson
Copy link
Contributor Author

Any other issues stopping the merge?

@mattyclarkson
Copy link
Contributor Author

Have resolved the previous issues in the pull request and squashed the commits as requested.

@sschuberth
Copy link
Contributor

Ok, final nit-pick: Please change the commit message from "Added a BATCH file that attempts to authorize SSH keys with the ssh-agent." to "cmd: Add a .bat file that attempts to authorize SSH keys with ssh-agent" (note there's not dot at the end of the sentence). I'll merge then right away. Thanks!

@dscho
Copy link
Member

dscho commented Nov 26, 2013

@mattyclarkson could you make that last change, please? Thank you!

If double clicked on from explorer it will start a new command prompt but
invoking the script from a current command prompt just adds the ssh keys
to the prompt.
@mattyclarkson
Copy link
Contributor Author

Done. Sorry for the slow response have been really ill recently and not in reach of a computer.

sschuberth added a commit that referenced this pull request Nov 27, 2013
BATCH file that starts the ssh-agent
@sschuberth sschuberth merged commit cb31665 into msysgit:master Nov 27, 2013
@dscho
Copy link
Member

dscho commented Nov 27, 2013

@mattyclarkson no need for apologies (but I am sorry that you have been ill, and hope you feel better now). Thanks for your contribution!

@mattyclarkson
Copy link
Contributor Author

Thanks. Hope it is useful, if there are any bug reports for it, copy me in.

On 27 November 2013 15:10, dscho [email protected] wrote:

@mattyclarkson https://github.com/mattyclarkson no need for apologies
(but I am sorry that you have been ill, and hope you feel better now).
Thanks for your contribution!


Reply to this email directly or view it on GitHubhttps://github.com//pull/135#issuecomment-29391332
.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants