aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGerald Combs <gerald@wireshark.org>2021-07-14 17:17:17 -0700
committerGerald Combs <gerald@wireshark.org>2021-07-14 17:33:18 -0700
commit9b78a42855cb778c9ad99cab017fbb9143afd923 (patch)
tree030a8f862946ac8099def1d012aafc9136a5efaf
parentf6872b0c3e4b81eca99087c15dd1449b64a70d65 (diff)
CMake: Fixup qtui's includes.
Add ${MINIZIP_INCLUDE_DIRS} to qtui's includes instead of every target's. Make more includes SYSTEM PRIVATE.
-rw-r--r--CMakeLists.txt8
-rw-r--r--ui/qt/CMakeLists.txt15
2 files changed, 6 insertions, 17 deletions
diff --git a/CMakeLists.txt b/CMakeLists.txt
index a101da1210..0a74bf8391 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -1281,13 +1281,6 @@ if(CMAKE_SYSTEM_NAME STREQUAL "Linux")
find_package(SETCAP)
endif()
-# Include minizip include directories
-if(MINIZIP_FOUND)
- include_directories(SYSTEM ${MINIZIP_INCLUDE_DIRS})
-else()
- set(MINIZIP_LIBRARY "")
-endif()
-
# Windows version updates
ws_find_package(WinSparkle ENABLE_WINSPARKLE HAVE_SOFTWARE_UPDATE)
@@ -2854,6 +2847,7 @@ if(BUILD_dumpcap AND PCAP_FOUND)
add_executable(dumpcap ${dumpcap_FILES})
set_extra_executable_properties(dumpcap "Executables")
target_link_libraries(dumpcap ${dumpcap_LIBS})
+ target_include_directories(dumpcap SYSTEM PRIVATE ${ZLIB_INCLUDE_DIRS})
install(TARGETS dumpcap
RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR}
PERMISSIONS ${DUMPCAP_SETUID}
diff --git a/ui/qt/CMakeLists.txt b/ui/qt/CMakeLists.txt
index 06d7af30f3..35c902c1d9 100644
--- a/ui/qt/CMakeLists.txt
+++ b/ui/qt/CMakeLists.txt
@@ -705,18 +705,13 @@ add_library(qtui OBJECT
)
target_include_directories(qtui
- SYSTEM PUBLIC
- ${QT_INCLUDE_DIRS}
- ${WINSPARKLE_INCLUDE_DIRS}
- ${SPEEXDSP_INCLUDE_DIRS}
-)
-
-target_include_directories(qtui
SYSTEM PRIVATE
- # Can't use imported target, linking object libraries for usage requirements
- # requires CMake 3.12
- ${PCAP_INCLUDE_DIRS}
${GCRYPT_INCLUDE_DIRS}
+ ${MINIZIP_INCLUDE_DIRS}
+ ${PCAP_INCLUDE_DIRS}
+ ${QT_INCLUDE_DIRS}
+ ${SPEEXDSP_INCLUDE_DIRS}
+ ${WINSPARKLE_INCLUDE_DIRS}
PRIVATE
${CMAKE_CURRENT_BINARY_DIR}
${CMAKE_CURRENT_SOURCE_DIR}