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

Fix "Unknown compiler version" warnings on MSVC #1049

Merged
merged 1 commit into from Feb 22, 2018
Merged

Conversation

jdonald
Copy link
Contributor

@jdonald jdonald commented Feb 22, 2018

Starting with MSVC 2017 we've been spamming the logs with this:

Unknown compiler version - please run the configure tests and report the results

which comes from boost (autoboost). However, this warning is unhelpful because autoboost does not contain boost's "configure tests".

We could raise the threshold version number (1913 to be current, or higher) but that would defer to problem. Better to follow gcc.hpp and clang.hpp which do not emit warnings for untested compiler versions. When we do hit a compiler error, we'll know.

@jdonald jdonald added the bug label Feb 22, 2018
@jdonald jdonald added this to the v1.0.6 milestone Feb 22, 2018
@jdonald jdonald requested a review from wmisha February 22, 2018 19:16
Copy link
Contributor

@wmisha wmisha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, so that's where it came from. Yeah, no point for the compiler to speculate on potential errors.

@wmisha wmisha merged commit 74760e7 into master Feb 22, 2018
@wmisha wmisha deleted the fix-unknown-compiler branch February 22, 2018 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants