Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1511)

Issue 91010050: code review 91010050: go.crypto/ssh/agent: fix darwin test failures.

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 11 months ago by dfc
Modified:
9 years, 8 months ago
Reviewers:
agl1, jmp, hanwen-google, gobot
CC:
golang-codereviews
Visibility:
Public.

Description

go.crypto/ssh/agent: fix darwin test failures. Darwin lacks ssh-agent support for ECDSA keys and RSA certs.

Patch Set 1 #

Patch Set 2 : diff -r 5478be1963aa https://code.google.com/p/go.crypto #

Patch Set 3 : diff -r 5478be1963aa https://code.google.com/p/go.crypto #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -0 lines) Patch
M ssh/agent/client_test.go View 1 2 chunks +15 lines, -0 lines 0 comments Download

Messages

Total messages: 7
dfc
Hello hanwen@google.com, jonathan.mark.pittman@gmail.com, agl@golang.org (cc: golang-codereviews@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go.crypto
9 years, 11 months ago (2014-05-02 17:40:07 UTC) #1
dfc
ping
9 years, 11 months ago (2014-05-05 03:01:51 UTC) #2
jmp
On 2014/05/05 03:01:51, dfc wrote: > ping Does the built-in Darwin agent support certs at ...
9 years, 11 months ago (2014-05-05 05:10:51 UTC) #3
hanwen-google
Is there a way to make this less ad-hoc? Maybe using "ssh-add" inserting keys to ...
9 years, 11 months ago (2014-05-05 09:24:01 UTC) #4
jmp
Darwin Aeschylus.local 13.1.0 Darwin Kernel Version 13.1.0: Wed Apr 2 23:52:02 PDT 2014; root:xnu-2422.92.1~2/RELEASE_X86_64 x86_64 ...
9 years, 11 months ago (2014-05-06 02:57:50 UTC) #5
jmp
On 2014/05/06 02:57:50, jmp wrote: > Darwin Aeschylus.local 13.1.0 Darwin Kernel Version 13.1.0: Wed Apr ...
9 years, 11 months ago (2014-05-06 03:00:29 UTC) #6
gobot
9 years, 8 months ago (2014-08-19 01:52:25 UTC) #7
R=close (assigned by dave@cheney.net)
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b