From 719d09d0cd392fcd19e04844df200895d3cfa70d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Fl=C3=B6ser?= Date: Fri, 28 Jul 2017 07:00:56 +0200 Subject: [PATCH] -Wno-inconsistent-missing-override on Clang This is a totally stupid warning for an older code base such as KWin. It generates hundreds of warnings as on legacy code no method uses override. The output is totally spammed, so it's better to disable it. This will hopefully also prevent that someone tries to fix it again by changing all of KWin - which we don't want. --- CMakeLists.txt | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/CMakeLists.txt b/CMakeLists.txt index b28e8ffd4c..9d2944805e 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -51,6 +51,10 @@ add_definitions(-DQT_DISABLE_DEPRECATED_BEFORE=0 -DQT_USE_QSTRINGBUILDER) set(CMAKE_CXX_STANDARD 14) set(CMAKE_CXX_STANDARD_REQUIRED ON) +if("${CMAKE_CXX_COMPILER_ID}" STREQUAL "Clang") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-inconsistent-missing-override") +endif() + find_package(Qt5Multimedia QUIET) set_package_properties(Qt5Multimedia PROPERTIES PURPOSE "Runtime-only dependency for effect video playback"