-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
Linking error with MSVC build #29479
Comments
Most testing of MSVC so far has happened with VS 2013/2015, so that's probably where this error is originating from. Thanks for the report, though! I'm curious what happened to these functions between VS 2012 and now... |
These are all C99 functions, VC++ didn't supported them before they became part of C++11 |
That makes sense. Perhaps it would be prudent to include a note in https://www.rust-lang.org/downloads.html#win-foot that VS 2013 is a minimum required version. Unless you think the work involved to fix this in VS2012 is trivial, I'd be fine if this issue was closed as a documentation bug. |
I'd also be totally on board with backcompat to VS2012 and prior if it's possible, but it looks like these functions just weren't provided, so it may not be so trivial. I'll tag this as docs for now to ensure we document that VS 2013 is needed. |
@alexcrichton Sadly we would need openlibm for backcompat to VS 2012. Although if we did use openlibm for math, that would give us backcompat all the way to VC6 |
I think this is a duplicate #29126. But given that the 1.4 release notes says the MSVC ABI build is "supported and recommended for use", I expected this to work out-of-the-box, so I'm opening a new issue to raise visibility:
The Rust 1.4 stable MSVC builds (and the 1.5 beta MSVC builds) fail to link a trivial hello world. #29126 suggests a workaround (use
-O -C lto
), but I feel this shouldn't be needed.I can reproduce this issue with Visual Studio 2012, Visual Studio 2010, and Visual Studio 2008.
Thanks!
The text was updated successfully, but these errors were encountered: