Skip to content
This repository has been archived by the owner on May 3, 2019. It is now read-only.

Commit

Permalink
Use SYSTEM include directories for 3rdparties include dirs
Browse files Browse the repository at this point in the history
Hopefully this will fix warnings coming from Qt with GCC8 (see #193)
  • Loading branch information
ColinDuquesnoy committed Feb 7, 2018
1 parent f7e5be4 commit 7e9bd48
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 15 deletions.
2 changes: 1 addition & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ if (UNIX AND NOT APPLE)
endif()

include(cmake/FindDependencies.cmake)
include_directories(3rdparty)
include_directories(SYSTEM 3rdparty)
# Sources
if(ENABLE_COVERAGE)
include(cmake/CodeCoverage.cmake)
Expand Down
26 changes: 13 additions & 13 deletions cmake/FindDependencies.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -16,19 +16,19 @@ find_package(Qt5 ${QT_MIN_VERSION} CONFIG REQUIRED COMPONENTS
WebEngineWidgets
Widgets)
find_package(Qt5LinguistTools REQUIRED)
include_directories(${Qt5Core_INCLUDE_DIRS})
include_directories(${Qt5Concurren_INCLUDE_DIRS})
include_directories(${Qt5Gui_INCLUDE_DIRS})
include_directories(${Qt5Network_INCLUDE_DIRS})
include_directories(${Qt5Qml_INCLUDE_DIRS})
include_directories(${Qt5Quick_INCLUDE_DIRS})
include_directories(${Qt5QuickControls2_INCLUDE_DIRS})
include_directories(${Qt5Sql_INCLUDE_DIRS})
include_directories(${Qt5Svg_INCLUDE_DIRS})
include_directories(${Qt5Test_INCLUDE_DIRS})
include_directories(${Qt5WebEngine_INCLUDE_DIRS})
include_directories(${Qt5WebEngineWidgets_INCLUDE_DIRS})
include_directories(${Qt5Widgets_INCLUDE_DIRS})
include_directories(SYSTEM ${Qt5Core_INCLUDE_DIRS})
include_directories(SYSTEM ${Qt5Concurren_INCLUDE_DIRS})
include_directories(SYSTEM ${Qt5Gui_INCLUDE_DIRS})
include_directories(SYSTEM ${Qt5Network_INCLUDE_DIRS})
include_directories(SYSTEM ${Qt5Qml_INCLUDE_DIRS})
include_directories(SYSTEM ${Qt5Quick_INCLUDE_DIRS})
include_directories(SYSTEM ${Qt5QuickControls2_INCLUDE_DIRS})
include_directories(SYSTEM ${Qt5Sql_INCLUDE_DIRS})
include_directories(SYSTEM ${Qt5Svg_INCLUDE_DIRS})
include_directories(SYSTEM ${Qt5Test_INCLUDE_DIRS})
include_directories(SYSTEM ${Qt5WebEngine_INCLUDE_DIRS})
include_directories(SYSTEM ${Qt5WebEngineWidgets_INCLUDE_DIRS})
include_directories(SYSTEM ${Qt5Widgets_INCLUDE_DIRS})


if (UNIX AND NOT APPLE)
Expand Down
2 changes: 1 addition & 1 deletion tests/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
include_directories(SYSTEM ${CMAKE_SOURCE_DIR}/3rdparty/catch)
include_directories(SYSTEM ${CMAKE_SOURCE_DIR}/3rdparty/fakeit)
include_directories(${CMAKE_SOURCE_DIR}/3rdparty/gtest-gmock)
include_directories(SYSTEM ${CMAKE_SOURCE_DIR}/3rdparty/gtest-gmock)
include_directories(../src/lib)
include_directories(.)
include_directories(Lib)
Expand Down

0 comments on commit 7e9bd48

Please sign in to comment.