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

OpenSSL v0.7.14 breaks build on Windows #566

Closed
kwaegel opened this issue Oct 10, 2016 · 3 comments
Closed

OpenSSL v0.7.14 breaks build on Windows #566

kwaegel opened this issue Oct 10, 2016 · 3 comments

Comments

@kwaegel
Copy link
Contributor

kwaegel commented Oct 10, 2016

Not sure if this is possible to fix on our side, but the recursive dependency on OpenSSL v0.7.14 is breaking the windows build with the error log attached below.

This appears to be an old bug in rust-openssl, since building v0.7.14 causes the same error, but building v0.8.0 or newer works fine.

Is it possible to upgrade this locally, or do we need to wait for our dependencies to upgrade? I'm not entirely certain how Cargo works here.

The following dependencies currently depend on openssl v0.7.14: cookie, hyper, openssl-verify.

error: failed to run custom build command for `openssl v0.7.14`
process didn't exit successfully: `C:\Users\[****]\dev\rust-rosetta\target\debug\build\openssl-5464f8f6e728c35a\build-script-build` (exit code: 101)
--- stdout
TARGET = Some("x86_64-pc-windows-msvc")
OPT_LEVEL = Some("0")
PROFILE = Some("debug")
TARGET = Some("x86_64-pc-windows-msvc")
debug=true opt-level=0
HOST = Some("x86_64-pc-windows-msvc")
TARGET = Some("x86_64-pc-windows-msvc")
TARGET = Some("x86_64-pc-windows-msvc")
HOST = Some("x86_64-pc-windows-msvc")
CC_x86_64-pc-windows-msvc = None
CC_x86_64_pc_windows_msvc = None
HOST_CC = None
CC = None
TARGET = Some("x86_64-pc-windows-msvc")
HOST = Some("x86_64-pc-windows-msvc")
CFLAGS_x86_64-pc-windows-msvc = None
CFLAGS_x86_64_pc_windows_msvc = None
HOST_CFLAGS = None
CFLAGS = None
running: "C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\bin\\amd64\\cl.exe" "/nologo" "/MD" "/Z7" "/I" "C:\\OpenSSL-Win64\\include" "/FoC:\\Users\\[****]\\dev\\rust-rosetta\\target\\debug\\build\\openssl-5464f8f6e728c35a\\out\\src\\c_helpers.o" "/c" "src/c_helpers.c"
c_helpers.c
C:\OpenSSL-Win64\include\openssl/lhash.h(198): warning C4090: 'function': different 'const' qualifiers
src/c_helpers.c(4): error C2037: left of 'references' specifies undefined struct/union 'ssl_st'
src/c_helpers.c(4): error C2065: 'CRYPTO_LOCK_SSL': undeclared identifier
src/c_helpers.c(8): error C2037: left of 'references' specifies undefined struct/union 'ssl_ctx_st'
src/c_helpers.c(8): error C2065: 'CRYPTO_LOCK_SSL_CTX': undeclared identifier
src/c_helpers.c(12): error C2037: left of 'references' specifies undefined struct/union 'x509_st'
src/c_helpers.c(12): error C2065: 'CRYPTO_LOCK_X509': undeclared identifier
ExitStatus(ExitStatus(2))
@Hoverbear
Copy link
Collaborator

Hm, I think we might need to wait for them to update... What happens if you blow away the Cargo.locks on the tasks which use these? (Can probably just blow away all Cargo.locks to test.)

@Hoverbear
Copy link
Collaborator

Looks like hyper still depend son 0.7: https://github.com/hyperium/hyper/blob/master/Cargo.toml#L32

@kwaegel
Copy link
Contributor Author

kwaegel commented Oct 10, 2016

hyper already has a bug open for this: hyperium/hyper#907

I dug a little further, and even if hyper upgrades, cookie hasn't released a version that depends on openssl v0.8.0 yet (though the change is already in the dev tree).

Looks like we'll just have to wait for now.

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

No branches or pull requests

2 participants