-
Notifications
You must be signed in to change notification settings - Fork 78
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
fix(Windows): unbreak Windows build #12840
Conversation
a feeble attempt to make M$ compiler grok otherwise valid C++17 code
Jenkins Builds
|
20282ba
to
ba752fc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but it would be nice to force msvc to accept it somehow if possible :)
Yeah, feel free to experiment with the cmake options/switches... I failed :/ Maybe the MSVC compiler version is simply too old, dunno. What version are we using here @jakubgs ? |
@caybro you can look those things up yourself in: |
Thanks, seems to be "14.29.30133" |
a feeble attempt to make M$ compiler grok otherwise valid C++17 code
What does the PR do
Unbreaks Windows build