-
Notifications
You must be signed in to change notification settings - Fork 41
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
Optional openssl #70
Comments
We don't have it as a direct dependency though. |
If your system doesn't have OpenSSL installed, |
This would need a fix in the upstream projects then. |
Indeed, from my quick checks it seems it is either hf-hub or some other package which is using it for downloading data from the internet with reqwest or an analogue, these packages already support rustls so it should be possible to make some simple upstream changes. |
It is being pulled in by |
It may be added back though huggingface/hf-hub#56 |
OpenSSL can often be annoying to use as a build dependency and it should be possible to use the
rustls
package to replaceopenssl
The text was updated successfully, but these errors were encountered: