Skip to content

Commit c05817f

Browse files
committed
build system: remove zlib-related code
As zlib is not used in the CPP-Rust Driver at all (it was used in CPP Driver for DataStax Cloud functionalities), this commit removes all references to zlib from the build system.
1 parent 38baef8 commit c05817f

File tree

7 files changed

+0
-168
lines changed

7 files changed

+0
-168
lines changed

CMakeLists.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,6 @@ option(CASS_USE_LIBSSH2 "Use libssh2 for integration tests" OFF)
4747
option(CASS_USE_OPENSSL "Use OpenSSL" ON)
4848
option(CASS_USE_STATIC_LIBS "Link static libraries when building executables" OFF)
4949
option(CASS_USE_STD_ATOMIC "Use std::atomic library" ON)
50-
option(CASS_USE_ZLIB "Use zlib" OFF)
5150
option(CASS_USE_TIMERFD "Use timerfd (Linux only)" ON)
5251
option(CASS_USE_LIBUV "Use libuv" OFF)
5352

cmake/Dependencies.cmake

Lines changed: 0 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -107,40 +107,6 @@ if(CASS_USE_OPENSSL)
107107
set(CASS_LIBS ${CASS_LIBS} ${OPENSSL_LIBRARIES})
108108
endif()
109109

110-
#------------------------
111-
# ZLIB
112-
#------------------------
113-
114-
if(CASS_USE_ZLIB)
115-
if(NOT ZLIB_LIBRARY_NAME)
116-
# Setup the root directory for zlib
117-
set(ZLIB_ROOT "${PROJECT_SOURCE_DIR}/lib/zlib/"
118-
"${PROJECT_SOURCE_DIR}/build/libs/zlib/")
119-
set(ZLIB_ROOT ${ZLIB_ROOT} ${ZLIB_ROOT_DIR} $ENV{ZLIB_ROOT_DIR})
120-
121-
# Ensure zlib was found (assign zlib include/libraries or present warning)
122-
find_package(ZLIB)
123-
if(ZLIB_FOUND)
124-
# Determine if the static library needs to be used for Windows
125-
if(WIN32 AND CASS_USE_STATIC_LIBS)
126-
string(REPLACE "zlib.lib" "zlibstatic.lib" ZLIB_LIBRARIES "${ZLIB_LIBRARIES}")
127-
endif()
128-
129-
# Assign zlib properties
130-
set(CASS_INCLUDES ${CASS_INCLUDES} ${ZLIB_INCLUDE_DIRS})
131-
set(CASS_LIBS ${CASS_LIBS} ${ZLIB_LIBRARIES})
132-
set(HAVE_ZLIB On)
133-
else()
134-
message(WARNING "Could not find zlib, try to set the path to zlib root folder in the system variable ZLIB_ROOT_DIR")
135-
message(WARNING "zlib libraries will not be linked into build")
136-
endif()
137-
else()
138-
# Assign zlib properties
139-
set(CASS_INCLUDES ${CASS_INCLUDES} ${ZLIB_INCLUDE_DIRS})
140-
set(CASS_LIBS ${CASS_LIBS} ${ZLIB_LIBRARIES})
141-
endif()
142-
endif()
143-
144110
#------------------------
145111
# Kerberos
146112
#------------------------

cmake/ExternalProject-OpenSSL.cmake

Lines changed: 0 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,6 @@ if(NOT OPENSSL_VERSION)
2525
set(OPENSSL_VERSION "1.0.2s")
2626
endif()
2727
option(OPENSSL_INSTALL_PREFIX "OpenSSL installation prefix location")
28-
if(CASS_USE_ZLIB)
29-
include(ExternalProject-zlib)
30-
endif()
3128
set(OPENSSL_VERSION ${OPENSSL_VERSION} CACHE STRING "OpenSSL version to build and install" FORCE)
3229

3330
# Determine the major and minor version of OpenSSL used
@@ -91,19 +88,6 @@ else()
9188
endif()
9289
set(OPENSSL_ROOT_DIR "${OPENSSL_INSTALL_DIR}" CACHE STRING "OpenSSL root directory" FORCE)
9390

94-
# Create build options for the platform build scripts
95-
if(BUILD_SHARED_LIBS)
96-
if(CASS_USE_ZLIB)
97-
set(OPENSSL_ZLIB_CONFIGURE_ARGUMENT "zlib-dynamic")
98-
set(ZLIB_LIB zlib.lib)
99-
endif()
100-
else()
101-
if(CASS_USE_ZLIB)
102-
set(OPENSSL_ZLIB_CONFIGURE_ARGUMENT "no-zlib-dynamic")
103-
set(ZLIB_LIB zlibstatic.lib)
104-
endif()
105-
endif()
106-
10791
# Determine if shared or static library should be built
10892
set(OPENSSL_CONFIGURE_COMPILER "no-asm no-ssl2")
10993
if(BUILD_SHARED_LIBS)
@@ -273,16 +257,6 @@ file(APPEND ${OPENSSL_CONFIGURE_SCRIPT}
273257
"POPD\r\n"
274258
"SET PATH=${PERL_PATH};%PATH%\r\n"
275259
"CALL :SHORTENPATH \"${OPENSSL_NATIVE_INSTALL_DIR}\" SHORTENED_OPENSSL_INSTALL_DIR\r\n")
276-
if(CASS_USE_ZLIB)
277-
# OpenSSL requires zlib paths to be relative (otherwise build errors may occur)
278-
externalproject_get_property(${OPENSSL_LIBRARY_NAME} SOURCE_DIR)
279-
file(RELATIVE_PATH ZLIB_INCLUDE_RELATIVE_DIR ${SOURCE_DIR} ${ZLIB_INCLUDE_DIR})
280-
file(TO_NATIVE_PATH ${ZLIB_INCLUDE_RELATIVE_DIR} ZLIB_NATIVE_INCLUDE_RELATIVE_DIR)
281-
file(RELATIVE_PATH ZLIB_LIBRARY_RELATIVE_DIR ${SOURCE_DIR} ${ZLIB_LIBRARY_DIR})
282-
file(TO_NATIVE_PATH ${ZLIB_LIBRARY_RELATIVE_DIR} ZLIB_NATIVE_LIBRARY_RELATIVE_DIR)
283-
set(OPENSSL_WITH_ZLIB_ARGUMENT "zlib ${OPENSSL_ZLIB_CONFIGURE_ARGUMENT} --with-zlib-include=\"${ZLIB_NATIVE_INCLUDE_RELATIVE_DIR}\" --with-zlib-lib=\"${ZLIB_NATIVE_LIBRARY_RELATIVE_DIR}\\${ZLIB_LIB}\"")
284-
set(OPENSSL_WITH_ZLIB_ARGUMENT "zlib ${OPENSSL_ZLIB_CONFIGURE_ARGUMENT} --with-zlib-include=\"${ZLIB_INCLUDE_RELATIVE_DIR}\" --with-zlib-lib=\"${ZLIB_LIBRARY_RELATIVE_DIR}\\${ZLIB_LIB}\"")
285-
endif()
286260
file(APPEND ${OPENSSL_CONFIGURE_SCRIPT}
287261
"perl Configure ${OPENSSL_WITH_ZLIB_ARGUMENT} --openssldir=!SHORTENED_OPENSSL_INSTALL_DIR! --prefix=!SHORTENED_OPENSSL_INSTALL_DIR! ${OPENSSL_CONFIGURE_COMPILER}\r\n"
288262
"IF NOT %ERRORLEVEL% EQU 0 (\r\n"
@@ -317,10 +291,5 @@ file(APPEND ${OPENSSL_CONFIGURE_SCRIPT}
317291
" FOR %%A IN (\"%~1\") DO SET %~2=%%~SA\r\n"
318292
" EXIT /B\r\n")
319293

320-
# Determine if zlib should be added as a dependency
321-
if(CASS_USE_ZLIB)
322-
add_dependencies(${OPENSSL_LIBRARY_NAME} ${ZLIB_LIBRARY_NAME})
323-
endif()
324-
325294
# Update the include directory to use OpenSSL
326295
include_directories(${OPENSSL_INCLUDE_DIR})

cmake/ExternalProject-zlib.cmake

Lines changed: 0 additions & 99 deletions
This file was deleted.

driver_config.hpp.in

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,5 @@
1414
#cmakedefine HAVE_ARC4RANDOM
1515
#cmakedefine HAVE_GETRANDOM
1616
#cmakedefine HAVE_TIMERFD
17-
#cmakedefine HAVE_ZLIB
1817

1918
#endif

src/CMakeLists.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,6 @@ set(HAVE_BOOST_ATOMIC ${CASS_USE_BOOST_ATOMIC})
124124
set(HAVE_STD_ATOMIC ${CASS_USE_STD_ATOMIC})
125125
set(HAVE_KERBEROS ${CASS_USE_KERBEROS})
126126
set(HAVE_OPENSSL ${CASS_USE_OPENSSL})
127-
set(HAVE_ZLIB ${CASS_USE_ZLIB})
128127

129128
# Generate the driver_config.hpp file
130129
configure_file(

src/driver_config.hpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,5 @@
1414
/* #undef HAVE_ARC4RANDOM */
1515
#define HAVE_GETRANDOM
1616
#define HAVE_TIMERFD
17-
#define HAVE_ZLIB
1817

1918
#endif

0 commit comments

Comments
 (0)