From 28e5cd6b56742a2017becb0c245711950d6c4382 Mon Sep 17 00:00:00 2001 From: baldurk Date: Tue, 3 Jan 2017 14:23:59 +0000 Subject: [PATCH] Pass warning flags into qmake when building --- CMakeLists.txt | 8 ++++---- qrenderdoc/CMakeLists.txt | 2 +- qrenderdoc/qrenderdoc.pro | 2 +- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index e24fa1509..c0bcc976f 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -123,12 +123,12 @@ if(CMAKE_COMPILER_IS_GNUCXX OR CMAKE_CXX_COMPILER_ID MATCHES "Clang") list(APPEND warning_flags -Wno-unused-but-set-variable) endif() + if(NOT CMAKE_BUILD_TYPE STREQUAL "Release") + list(APPEND warning_flags -Werror) + endif() + string(REPLACE ";" " " warning_flags "${warning_flags}") set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${warning_flags}") - - if(NOT CMAKE_BUILD_TYPE STREQUAL "Release") - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Werror") - endif() endif() if(ANDROID) diff --git a/qrenderdoc/CMakeLists.txt b/qrenderdoc/CMakeLists.txt index 74fac5d58..16afb23e9 100644 --- a/qrenderdoc/CMakeLists.txt +++ b/qrenderdoc/CMakeLists.txt @@ -18,7 +18,7 @@ endif() # which causes CMake to rerun this target every time so that Qt # can do dependency checking and rebuild anything necessary. add_custom_command(OUTPUT QRenderDoc - COMMAND qmake "CONFIG+=${QMAKE_CONFIG}" "QMAKE_LFLAGS+=${QMAKE_LDFLAGS}" "DEFINES+=GIT_COMMIT_HASH_LITERAL=${GIT_COMMIT_HASH}" "DESTDIR=${output_dir}" ${CMAKE_CURRENT_SOURCE_DIR} + COMMAND qmake "CONFIG+=${QMAKE_CONFIG}" "QMAKE_CXXFLAGS+=${warning_flags}" "QMAKE_LFLAGS+=${QMAKE_LDFLAGS}" "DEFINES+=GIT_COMMIT_HASH_LITERAL=${GIT_COMMIT_HASH}" "DESTDIR=${output_dir}" ${CMAKE_CURRENT_SOURCE_DIR} COMMAND MAKEFLAGS= make --no-print-directory) add_custom_target(build-qrenderdoc ALL DEPENDS QRenderDoc DEPENDS renderdoc) diff --git a/qrenderdoc/qrenderdoc.pro b/qrenderdoc/qrenderdoc.pro index 5c68fcdf1..ce7b0ccab 100644 --- a/qrenderdoc/qrenderdoc.pro +++ b/qrenderdoc/qrenderdoc.pro @@ -77,7 +77,7 @@ win32 { LIBS += -L$$DESTDIR -lrenderdoc QMAKE_LFLAGS += '-Wl,-rpath,\'\$$ORIGIN\'' - QMAKE_CXXFLAGS += -std=c++11 -Wno-unused-parameter -Wno-reorder + QMAKE_CXXFLAGS += -std=c++11 macx: { DEFINES += RENDERDOC_PLATFORM_POSIX RENDERDOC_PLATFORM_APPLE