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.

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

Update libssh2-sys #4008

Merged
merged 1 commit into from
May 8, 2017
Merged

Update libssh2-sys #4008

merged 1 commit into from
May 8, 2017

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
Collaborator

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
Collaborator

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
Collaborator

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