Share ssh-agent instances across rxvt instances

RESOLVED FIXED

Status

mozilla.org
MozillaBuild
RESOLVED FIXED
11 years ago
10 years ago

People

(Reporter: Ben Turner (not reading bugmail, use the needinfo flag!), Assigned: Ben Turner (not reading bugmail, use the needinfo flag!))

Tracking

(Blocks: 1 bug)

Details

Attachments

(1 attachment)

Created attachment 293962 [details] [diff] [review]
Patch

Right now there are two instances of ssh-agent that get created per instance of rxvt. Attached patch uses a script that I found here: 

http://www.cygwin.com/ml/cygwin/2001-06/msg00537.html
Attachment #293962 - Flags: review?(ted.mielczarek)
I guess the bad thing about this is that the single instance of ssh-agent will live forever (until shutdown)... but it's pretty small so I dont think anyone will really care. This makes it so that you only have to enter your password once per login session.
Comment on attachment 293962 [details] [diff] [review]
Patch

>+  ps -ef | grep ${SSH_AGENT_PID} | grep ssh-agent$ > /dev/null || {

FWIW, $SSH_AGENT_PID will work on MSYS afaict. Probably doesn't matter but I just wanted to mention it.
I emailed the script author asking for licensing permission.
Component: Build & Release → MozillaBuild
QA Contact: build → mozillabuild
Assignee: nobody → bent.mozilla
Comment on attachment 293962 [details] [diff] [review]
Patch

r+ but pending licensing I guess.  :-/
Attachment #293962 - Flags: review?(ted.mielczarek) → review+

Comment 5

11 years ago
This patch doesn't fix the newly born start-l10n.bat, too.
That file looks ok to me... I'm removing the ssh-agent lines from the others and start-l10n.bat doesn't have any reference to ssh-agent.

Comment 7

11 years ago
Hrm. That seems to be a double merge error. Myk landed ssh-agent support for all but one start script, start-msvc9.bat, which I used to create start-l10n.bat, for no particular reason.

Sounds like a good idea to sync all the start scripts in this regard.
That's what this patch will do - I'm backing out Myk's ssh-agent additions and then adding a profile script to do the real ssh-agent work. You got lucky :)

Comment 9

11 years ago
Gosh, I blame the elves for me not reading. Sorry.
Ted, Joseph Reagle contacted me via email and released the script under the GPL. Need a new patch or can I just add that on checkin?
Just add it, that's fine.
Fixed on trunk.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
Mass re-assign of MozillaBuild bugs into mozilla.org:MozillaBuild
Blocks: 418655
You need to log in before you can comment on or make changes to this bug.