D5489: Sanitizers: Don't use GCC-like flags for e.g. MSVC

2017-04-21 Thread Kevin Funk
This revision was automatically updated to reflect the committed changes. Closed by commit R240:c56cf46b4beb: Sanitizers: Don't use GCC-like flags for e.g. MSVC (authored by kfunk). REPOSITORY R240 Extra CMake Modules CHANGES SINCE LAST UPDATE

D5489: Sanitizers: Don't use GCC-like flags for e.g. MSVC

2017-04-18 Thread Ben Cooksley
bcooksley accepted this revision. bcooksley added a comment. Looks good to me - thanks for taking this on, it'll make getting CI working for Windows much easier. REPOSITORY R240 Extra CMake Modules BRANCH master REVISION DETAIL https://phabricator.kde.org/D5489 To: kfunk, aacid,

D5489: Sanitizers: Don't use GCC-like flags for e.g. MSVC

2017-04-18 Thread Albert Astals Cid
aacid accepted this revision. aacid added a comment. This revision is now accepted and ready to land. Makes sense, the other option would be actually erroring out as we do in check_compiler_version if you don't run "the correct clang/gcc", but i guess this may be enough for MSVC at this

D5489: Sanitizers: Don't use GCC-like flags for e.g. MSVC

2017-04-18 Thread Kevin Funk
kfunk added reviewers: aacid, bcooksley. REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D5489 To: kfunk, aacid, bcooksley Cc: #frameworks, #build_system

D5489: Sanitizers: Don't use GCC-like flags for e.g. MSVC

2017-04-18 Thread Kevin Funk
kfunk created this revision. Restricted Application added projects: Frameworks, Build System. Restricted Application added subscribers: Build System, Frameworks. REVISION SUMMARY Fixes compiler warnings such as: cl : Command line warning D9002 : ignoring unknown option