Skip to content

Conversation

@orf
Copy link
Contributor

@orf orf commented Aug 23, 2019

git2 is used to only display the current branch. We can get rid of the quite expensive libssh2-sys dependency by disabling the default features, which are only needed if you're cloning repos over ssh or https.

@sophiajt
Copy link
Contributor

Thanks! Good catch

@sophiajt sophiajt merged commit cd1e16d into nushell:master Aug 23, 2019
@orf orf deleted the git2-fix branch August 23, 2019 23:53
@lilyball
Copy link

Now that this has been merged, the advice on installing openssl can be removed from the book.

@est31
Copy link
Contributor

est31 commented Aug 24, 2019

@lilyball not for linux because it's still used there through the native-tls crate. On other OS's it might have become redundant.

@lilyball
Copy link

It's definitely redundant on macOS now. I was able to build nushell with a Nix derivation on macOS without including openssl.

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