-
Notifications
You must be signed in to change notification settings - Fork 120
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
wrap CMP0091 in MSVC check #1580
Changes from all commits
35ab1f4
f010e24
4bc2db0
876cc4d
d65887a
9411c24
a4bf13d
7e88dbf
0501779
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,6 +5,12 @@ if(MSVC) | |
endif() | ||
|
||
add_compile_options(/W4 ${WARNINGS_AS_ERRORS_FLAG} /wd5031 /wd4668 /wd4820 /wd4255 /wd4710 /analyze) | ||
|
||
add_compile_options( | ||
$<$<CONFIG:>:/MT> #---------| | ||
$<$<CONFIG:Debug>:/MTd> #---|-- Statically link the runtime libraries | ||
$<$<CONFIG:Release>:/MT> #--| | ||
) | ||
Comment on lines
+9
to
+13
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @barcharcraz and @BillyONeal - can you please help review and sign-off on this approach. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Just for reference I pulled it from here: |
||
elseif(CMAKE_C_COMPILER_ID MATCHES "Clang") | ||
if(WARNINGS_AS_ERRORS) | ||
set(WARNINGS_AS_ERRORS_FLAG "-Werror") | ||
|
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.
Consider including this fix in a changelog entry.
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.
Done in #1605
Thanks @danewalton