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

Bump reqwest dependency version to fix openssl build error #4

Merged
merged 1 commit into from
Dec 3, 2018

Conversation

mogenson
Copy link
Contributor

Current master branch with rustc 1.30 fails to build with:

error: failed to run custom build command for `openssl v0.9.24`                     
process didn't exit successfully: `/tmp/cargo-installg2QM2v/release/build/openssl-1ff837db13969d3e/build-script-build` (exit code: 101)
--- stderr
thread 'main' panicked at 'Unable to detect OpenSSL version', /home/mike/.cargo/registry/src/github.aaakk.us.kg-1ecc6299db9ec823/openssl-0.9.24/build.rs:16:14
note: Run with `RUST_BACKTRACE=1` for a backtrace.

warning: build failed, waiting for other jobs to finish...
error: failed to compile `hodl-ticker v1.5.2`, intermediate artifacts can be found at `/tmp/cargo-installg2QM2v`

Caused by:
  build failed

Reqwest version 0.8.5 pulls in native-tls version 0.1.5 which pulls in
openssl 0.9.24 which doesn't build on machines with OpenSSL 1.1.

See openssl issue here: sfackler/rust-openssl#994

Bump reqwest to 0.9 to pull in native-tls 0.2.2 and openssl 0.10.15 to
fix the build error.

reqwest version 0.8.5 pulls in native-tls version 0.1.5 which pulls in
openssl 0.9.24 which doesn't build on machines with OpenSSL 1.1.

Bump reqwest to 0.9 to pull in native-tls 0.2.2 and openssl 0.10.15 to
fix the build error.
@kooparse
Copy link
Owner

kooparse commented Dec 3, 2018

Hi @mogenson,

Thanks for your fix! 👍

@kooparse kooparse merged commit df3ca8b into kooparse:master Dec 3, 2018
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.

2 participants