diff options
-rw-r--r-- | CMakeLists.txt | 14 | ||||
-rw-r--r-- | ChangeLog | 4 |
2 files changed, 1 insertions, 17 deletions
diff --git a/CMakeLists.txt b/CMakeLists.txt index 217d48a..fce0cf9 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -319,19 +319,7 @@ endif() # if(URIPARSER_WARNINGS_AS_ERRORS) if(MSVC) - # NOTE: We cannot use global "add_definitions(/WX)" currently because: - # - it would affect target "testrunner" - # - testrunner depends on googletest - # - googletest is not free of warnings - # - CMake's "target_include_directories([..] SYSTEM PRIVATE [..])" - # is not showing any effect with MSVC - # - The equivalent of Clang's "-isystem <includedir>" is - # marked as experimental and a dead-end street according to - # https://developercommunity.visualstudio.com/t/fully-support-c-externali-flag-instead-of-making-i/1191080 - # so that is not used, either. - # As a result, warnings are fatal for all target but "testrunner" as of today. - target_compile_definitions(uriparser PRIVATE /WX) # the library - target_compile_definitions(uriparse PRIVATE /WX) # the CLI + add_definitions(/WX) else() set(URIPARSER_EXTRA_COMPILE_FLAGS "${URIPARSER_EXTRA_COMPILE_FLAGS} -Werror") endif() @@ -8,10 +8,6 @@ XXXX-XX-XX -- X.X.X CMAKE_CXX_* variables (GitHub #110) Thanks to Alexander Richardson for the patch (originally at libexpat) * Fixed: Windows: Address MSVC compiler warnings (GitHub #111) - * Improved: Windows: Exclude target "testrunner" from - -DURIPARSER_WARNINGS_AS_ERRORS=ON because dependency googletest - is not free of warnings and MSVC seems to lack an equivalent - of Clang's -isystem that is not marked experimental (GitHub #110) 2021-03-18 -- 0.9.5 |