-
Notifications
You must be signed in to change notification settings - Fork 708
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
cargo build on Windows doesn't work out of the box #12
Comments
I think this is due to a known problem with Cargo: rust-lang/cargo#1889. If you run in the "VS2015 x64 Native Tools Command Prompt" or the "VS2015 x85 Native Tools Command Prompt" or use vcvarsall.bat to set up the environment before building ring, it should work. Could you please try that and report back? IMO, the best place to fix this issue is within Cargo. I will ask Alex about it. |
Yep, works fine if I use one of the vcvars bat files. |
Thanks. I will update the build documentation to indicate that, at least for now, that is required. Also, I'm going to keep working with the Cargo people to find an automatic solution. |
See rust-lang/rust#30516 and rust-lang/rust#30229. |
I think recent improvements in Cargo have made this work for cases where you're not cross-compiling. Could you please retest? Thanks! |
Closing this. I've been building with |
Hey all, what exactly is the fix for this. 2 years later and I think I'm running into the same issue even after |
It can't be the same issue because ring's build system doesn't use msbuild on anymore. |
I have VS 2015 installed, so it would be nice if ring could find where msbuild is.
The text was updated successfully, but these errors were encountered: