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

Add Visual Studio 2017 builds to AppVeyor #338

Closed
briansmith opened this issue Nov 8, 2016 · 11 comments
Closed

Add Visual Studio 2017 builds to AppVeyor #338

briansmith opened this issue Nov 8, 2016 · 11 comments

Comments

@briansmith
Copy link
Owner

[Note that Visual Studio “15” is the successor to Visual Studio 2015 and will probably have a different name; see https://www.visualstudio.com/en-us/news/releasenotes/vs15-relnotes.]

Let's support building the C/C++ components with Visual Studio “15” and using its linker.

The AppVeyor issue for Visual Studio “15” support is appveyor/ci#753, where they say they'll wait at least until there's a beta release. But, we'll take patches to support it in ring before then.

@radix
Copy link

radix commented Dec 29, 2016

For reference, it appears that this is now officially called "Visual Studio 2017", and it now has a release candidate available. https://www.visualstudio.com/vs/visual-studio-2017-rc/

@briansmith briansmith changed the title Support building and test in CI with Visual Studio “15” Add Visual Studio 2017 builds to AppVeyor Mar 14, 2017
@briansmith
Copy link
Owner Author

This may blocked on rustc/cargo adding support for VS2017. See rust-lang/rust#38584.

@sanmai-NL
Copy link
Contributor

@briansmith

This may blocked on rustc/cargo adding support for VS2017. See rust-lang/rust#38584.

It seems that issue’s been resolved. Also, I didn’t have problems compiling a crate that uses ring 0.13alpha on an instance of the Windows 10 development VM, which seems to only contain Visual Studio 2017. So it now comes down to modifying the AppVeyor config, right?

@davlia
Copy link

davlia commented Jun 6, 2018

Currently still experiencing build issues on Windows 10 using VS 2017 build toolchain

@o0Ignition0o
Copy link

Sorry if I'm necrobumping or adding noise to the issue, but I've done a lot of testing today, and it turns out (as advised in the mozilla Building Firefox for Windows page) I'm able to build ring with a Visual Studio 2017 Update 6 (15.6) version, but not with the 15.7 version.

@davlia you might want to try this out, then again, YMMV :)

@trsh
Copy link

trsh commented Jun 24, 2018

15.7.4 fails :/

@davlia
Copy link

davlia commented Jun 24, 2018

Manage to fix the issue by setting some environment variables set CL="/wd5045". Also verify that your rust triple is correct: x86_64-pc-windows-msvc. @trsh

@trsh
Copy link

trsh commented Jun 25, 2018

@davlia your solution is very abstract. Can you provide some insight about the envs etc?

@szczurowski
Copy link

On my 15.7.4 the build fails :( Although I set the CL env-var.
BTW: here's a bit info about linker:
:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.14.26428\bin\Hostx64\x64>cl
Microsoft (R) C/C++ Optimizing Compiler Version 19.14.26431 for x64
Copyright (C) Microsoft Corporation. All rights reserved.

@dmitry-vorobiev
Copy link

dmitry-vorobiev commented Jul 21, 2018

set CL="/wd5045"

Thanks, it helped me))

I ran into the same issue with VS 15.7.5

@briansmith
Copy link
Owner Author

Probably not going to add Visual Studio 2017 coverage given 2019 has been out for a while, and due to CI/CD constraints.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants