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

Update support baseline #112

Merged
merged 4 commits into from
Aug 9, 2023
Merged

Update support baseline #112

merged 4 commits into from
Aug 9, 2023

Conversation

tstenner
Copy link
Collaborator

The support baseline was last touched in 2020. By now, several vendors have stopped supporting their components.
The new versions are mostly what is shipped by Ubuntu 20.04.

I would prefer to target MSVC 2019, but I suspect our industry representative ;-) @mgrivich will insist on MSVC 2017.

@tstenner
Copy link
Collaborator Author

Also pinging @chkothe.

@mgrivich
Copy link

I don't speak for all of industry, but Neurobehavioral Systems is currently moving from VS2019 to VS2022 for all Windows projects. I maintain an independent Visual Studio project because CMake annoys me.

If you are going to support one and only one version of Visual Studio, I'd target 3-5 years old, so VS2019 is the best. This would lead to the easiest integration with the largest number of projects.

@tstenner
Copy link
Collaborator Author

@mgrivich Great to hear this, what I last remembered was that you had to support some ancient windows versions newer C++ compilers couldn't target any more. I also had the 3-5 years in the back of my head, but 2017 didn't seem to be six years ago already. Let's put 2019 in there unless someone objects.

@cboulay
Copy link
Collaborator

cboulay commented Jun 11, 2023

Approved.

@tstenner
Copy link
Collaborator Author

Any objections from @chkothe?

@tstenner tstenner merged commit be438c7 into master Aug 9, 2023
@tstenner tstenner deleted the tstenner-patch-1 branch August 9, 2023 19:29
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

Successfully merging this pull request may close these issues.

None yet

3 participants