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

VC2019 / Visual Studio project configs for v142 compiler toolset #2113

Closed
wants to merge 5 commits into from

Conversation

richardeakin
Copy link
Collaborator

As discussed in #2082, skipping VC2017 folders since Visual Studio 2019 / v142 is stable, freely available, and in my opinion much better than Visual Studio 2017 (and 2015).

@vinjn
Copy link
Contributor

vinjn commented Jul 10, 2019

Agree, also vs2019 compiles faster than vs2015.

@fei4xu
Copy link

fei4xu commented Jul 22, 2019

I would suggest we skip the vs project *.vxcprojt totally, since current cmake system works fine on Windows and it's cross platform.
VS2019 also has direct cmake support.

@richardeakin
Copy link
Collaborator Author

@fei4xu we may think about that some time in the future, but up until now (and currently), very few Windows users are building via cmake - they all depend on the included VS project files, so we'll continue to supply them until that changes. A large part of this decision is that cmake-generated visual studio project files are difficult to hand tune, and are generally setup pretty poor compared to those we make by hand.

Using cmake directly in Visual Studio is interesting, although my attempts to use it have shown that it is still quite experimental, especially with more complex project structures.

std::experimental::filesystem no longer exists from VS 2019 16.3.1 onwards, need to use std::filesystem which is only available if using C++17
It doesn't currently compile with msw v142, and it's been decided to remove this class for the next cinder version, in favor of using popular json libraries directly.
@vinjn
Copy link
Contributor

vinjn commented Oct 9, 2019

@richardeakin @andrewfb can we get this merged? My project was broken by std::experimental::filesystem

@richardeakin
Copy link
Collaborator Author

Yea, it's a bummer. I think these changes will make it into master branch first thing after the next release lands (not exactly sure the ETA of that, I know @andrewfb has some smallish things pending around getting a few deps built first, win32 samples fixed but it's otherwise on the way). In the meantime feel free to pull my branch, I'll keep it updated with latest master as needed. Or if it's a huge hassle we can probably push a temp branch to the main repo..

@richardeakin richardeakin added this to the 0.9.3 milestone Oct 15, 2019
paulhoux added a commit to paulhoux/Cinder that referenced this pull request Feb 9, 2020
@richardeakin
Copy link
Collaborator Author

Closing, superseded by #2138

@richardeakin richardeakin removed this from the 0.9.3 milestone Feb 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants