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

Visual Studio 2017 support #516

Closed
amerkoleci opened this issue Nov 22, 2018 · 5 comments
Closed

Visual Studio 2017 support #516

amerkoleci opened this issue Nov 22, 2018 · 5 comments

Comments

@amerkoleci
Copy link

Is your feature request related to a problem? Please describe.
There are no related problems

Describe the solution you'd like
Update from VS2015 to VS2017

Describe alternatives you've considered
Understand if VS2017 clang works well or better use MSVC toolkit

OS and backend
This is related to windows OS.

@shartte
Copy link
Contributor

shartte commented Nov 22, 2018

I have been using VS2017 with Filament without much issues so far, what in particular is this issue about?
Is it about the pre-built packages?

@amerkoleci
Copy link
Author

I mean compile filament using clang vs2017 not about usage

@shartte
Copy link
Contributor

shartte commented Nov 22, 2018

Yes, but that is already working... I am compiling filament using clang-cl and VS2017, using the instructions found on the main page.

@romainguy
Copy link
Collaborator

Our instructions use Visual Studio 2015 because that's what our CI environment offers. According to @shartte it works in Visual Studio 2017 as well (which makes sense, we don't use anything VS specific I believe). Please open a new bug with more specific information if it doesn't work in VS 2017. Thanks!

@amerkoleci
Copy link
Author

Okay, thank you!

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

3 participants