Skip to content

Commit fa0e8b8

Browse files
committed
minor build system cleanups
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
1 parent 828d759 commit fa0e8b8

File tree

4 files changed

+12
-5
lines changed

4 files changed

+12
-5
lines changed

CMakeLists.txt

+4
Original file line numberDiff line numberDiff line change
@@ -462,6 +462,10 @@ if(ENABLE_TESTING)
462462
add_definitions(-DQT_BUILD_INTERNAL)
463463
endif()
464464

465+
if(NOT WITH_ZLIB OR NOT ZLIB_FOUND)
466+
katie_definition(-DQT_NO_COMPRESS)
467+
endif()
468+
465469
if(NOT WTIH_EGL OR NOT EGL_FOUND)
466470
katie_definition(-DQT_NO_EGL)
467471
endif()

src/core/CMakeLists.txt

+8-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
add_definitions(-DQT_BUILD_CORE_LIB)
2-
set(EXTRA_CORE_LIBS ${ZLIB_LIBRARIES})
2+
set(EXTRA_CORE_LIBS)
33

44
set(CORE_PUBLIC_HEADERS
55
${CORE_PUBLIC_HEADERS}
@@ -306,6 +306,13 @@ if(WITH_GLIB2 AND GLIB2_FOUND)
306306
include_directories(${GLIB2_INCLUDE_DIR})
307307
endif()
308308

309+
if(WITH_ZLIB AND ZLIB_FOUND)
310+
set(EXTRA_CORE_LIBS
311+
${EXTRA_CORE_LIBS}
312+
${ZLIB_LIBRARIES}
313+
)
314+
include_directories(${ZLIB_INCLUDE_DIRS})
315+
endif()
309316

310317
if(${KATIE_PLATFORM} STREQUAL "linux")
311318
set(EXTRA_CORE_LIBS

src/svg/CMakeLists.txt

-2
Original file line numberDiff line numberDiff line change
@@ -60,8 +60,6 @@ if(WITH_ZLIB AND ZLIB_FOUND)
6060
${ZLIB_LIBRARIES}
6161
)
6262
include_directories(${ZLIB_INCLUDE_DIRS})
63-
else()
64-
katie_definition(-DQT_NO_COMPRESS)
6563
endif()
6664

6765
katie_generate_misc("${SVG_HEADERS}" QtSvg)

src/tools/uic/CMakeLists.txt

-2
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,6 @@ include_directories(
66
${CMAKE_BINARY_DIR}/privateinclude
77
${CMAKE_BINARY_DIR}/include/QtCore
88
${CMAKE_BINARY_DIR}/privateinclude/QtCore
9-
${CMAKE_BINARY_DIR}/include/QtXml
10-
${CMAKE_BINARY_DIR}/privateinclude/QtXml
119
${CMAKE_CURRENT_SOURCE_DIR}
1210
${CMAKE_CURRENT_SOURCE_DIR}/cpp
1311
${BOOTSTRAP_INCLUDES}

0 commit comments

Comments
 (0)