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 submodule url #461

Merged
merged 1 commit into from
Apr 28, 2015
Merged

Conversation

phatblat
Copy link
Member

Moved to github as of 2015-03-23. There have been some commits but no official release since the move, so no need to update the commit yet. The biggest gain from the switch is no more invalid certificates.

fatal: unable to access 'https://git.libssh2.org/libssh2.git/': SSL certificate problem: Invalid certificate chain
Clone of 'git://git.libssh2.org/libssh2.git' into submodule path 'External/libssh2' failed

old: https://git.libssh2.org/?p=libssh2.git;a=shortlog;js=1
new: https://github.com/libssh2/libssh2/commits/master

announcement: http://www.libssh2.org/mail/libssh2-devel-archive-2015-03/0029.shtml

Moved to github as of 2015-03-23
@joshaber
Copy link
Member

Oh cool! Even better, it looks like once we bump libgit2 we won't need OpenSSL at all: libgit2/libgit2#2997

joshaber added a commit that referenced this pull request Apr 28, 2015
@joshaber joshaber merged commit ac7be76 into libgit2:master Apr 28, 2015
@phatblat phatblat deleted the phatblat/pr/libssh2 branch April 28, 2015 14:41
@mdiep
Copy link
Contributor

mdiep commented Apr 28, 2015

I think it's still needed for iOS?

@phatblat
Copy link
Member Author

Oh nice! Does the elimination of OpenSSL remove the need for libssh2?

@joshaber
Copy link
Member

I 🍩 know the answer to either of those questions 😄

@phatblat
Copy link
Member Author

libssh2 depends on OpenSSL, mostly for the crypto stuff, I believe. Because of this, ObjectiveGit still has a transitive dependency on OpenSSL, which means the libgit2 change won't improve the iOS build times 😞 The OS X build gets pre-built libssh2 and openssl from brew.

I take @carlosmn's comment to mean there may eventually be a change that removes the libssh2 dependency.

There is still of course libssh2 which will still link against openssl, so we're not quite free of threading nonsense yet, but someone was trying to upstream a SecureTransport version.

@carlosmn
Copy link
Member

libssh2 is an optional dependency, you can build without it you want; but libssh2 may lose its OpenSSL dependency, which would make it easier to write threaded programs.

@phatblat
Copy link
Member Author

Found some work on a libssh2 fork adding support for SecureTransport https://github.com/keithduncan/libssh2/pull/2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants