-
Notifications
You must be signed in to change notification settings - Fork 448
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 Woes #612
Comments
Off the top of my head I'm not sure what's causing this, but I found a related issue on Github (that's currently unsolved). This is a shot in the dark, but the function name Since I'm kinda out of my depth here (having not really worked with ASM before), maybe @briansmith (having written ring) can chime in? |
If the function starts with is your target x86_64 macOS? Which C/C++ compiler are you using? Which version? |
I'm working on a MacOS yes. Latest OS - I'm not sure how I would check my c/c++ compiler to be honest. |
10.12.4 |
|
|
could you echo $INCLUDE? In particular, do you have OpenSSL's header files in the include path ahead of ring's same-named header files? |
blank also
|
It looks like your xcode/clang is even newer than the one I use on Travis CI, if I'm understanding the version numbers correctly. I guess it is possible that something broke in newer versions but I doubt it, because I think the code is correct wr.t. that declaration. The declaration is in openssl/aes.h, which is included at the top of aes.c. Maybe something is borked in your environment. Could you try Regardless, to reduce the chance of confusion between OpenSSL and ring headers further, I just committed briansmith/ring@ff71d52, which will be in the next ring release. |
still gets me the same ring error - error: failed to run custom build command for
|
Cargo Lock File
|
Thanks for jumping in @briansmith ! We do have some docs around setting up OpenSSL for Macs and there's an issue to verify if it's still up to date: #533 . As far as I know we don't need any special setup any more and this does sound like a ring issue. For completeness everything works for me with $ cc --version
Apple LLVM version 8.0.0 (clang-800.0.42.1)
Target: x86_64-apple-darwin15.6.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin on OSX 10.11.6. Which reminds me it's probably time to run an OSX update. |
@drusellers can you upgrade to Rusoto 0.24.0? The bug report shows it using Rusoto 0.23.0 which includes an older version of ring. Rusoto 0.24.0 should pull in the most recent version of ring which may fix this problem. 😄 |
Great Success!!! |
Thanks @matthewkmayer! |
Made an issue to update the RELEASING docs so we shouldn't run into an issue like this again. 😄 |
Is there any documentation about OpenSSL needs? I'm getting errors like
main.rs
OpenSSL
Build Output
Build Output
Any ideas would be much appreciated.
The text was updated successfully, but these errors were encountered: