Skip to content

Conversation

@alexcrichton
Copy link
Member

It now has an updated initialization routine to hopefully get ssh working on
Windows, with a fix for alexcrichton/git2-rs#202

It now has an updated initialization routine to hopefully get ssh working on
Windows, with a fix for alexcrichton/git2-rs#202
@alexcrichton
Copy link
Member Author

@bors: r+

@bors
Copy link
Contributor

bors commented May 8, 2017

📌 Commit 77b7612 has been approved by alexcrichton

@rust-highfive
Copy link

r? @brson

(rust_highfive has picked a reviewer for you, use r? to override)

@bors
Copy link
Contributor

bors commented May 8, 2017

⌛ Testing commit 77b7612 with merge a232aef...

bors added a commit that referenced this pull request May 8, 2017
Update libssh2-sys

It now has an updated initialization routine to hopefully get ssh working on
Windows, with a fix for alexcrichton/git2-rs#202
@jethrogb
Copy link
Contributor

jethrogb commented May 8, 2017

Is it possible to add a test to cargo that tries to use SSH repos? This is apparently a not-often tested code path that breaks often.

@alexcrichton
Copy link
Member Author

I'd certainly love to add such a test! It'd likely involve spinning up an ssh server, however, which doing so reliably and in a cross-platform manner I'm not sure how to do.

@jethrogb
Copy link
Contributor

jethrogb commented May 8, 2017

Yes, I was afraid of that. Talking to something remote is out of the question?

@alexcrichton
Copy link
Member Author

Unfortunately, yes.

@bors
Copy link
Contributor

bors commented May 8, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing a232aef to master...

@bors bors merged commit 77b7612 into rust-lang:master May 8, 2017
@alexcrichton alexcrichton deleted the update-ssh branch May 11, 2017 14:53
@alexcrichton alexcrichton added the relnotes Release-note worthy label May 12, 2017
@ehuss ehuss added this to the 1.19.0 milestone Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

relnotes Release-note worthy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants