]> git.ipfire.org Git - thirdparty/curl.git/commitdiff
cmake: drop redundant zlib var, rename function (internals)
authorViktor Szakats <commit@vsz.me>
Fri, 13 Sep 2024 03:24:25 +0000 (05:24 +0200)
committerViktor Szakats <commit@vsz.me>
Thu, 19 Sep 2024 10:56:22 +0000 (12:56 +0200)
- drop redundant internal variable `USE_ZLIB`, rely on `HAVE_LIBZ`.

- rename `optional_dependency()` -> `curl_dependency_option()`
  Make `grep 'option('` hit this option. Namespaced.
  It has a single use with `ZLIB`.

Closes #14918

CMake/Macros.cmake
CMake/curl-config.cmake.in
CMakeLists.txt

index d268667ff71c3161dcb4ad1901cad2a5c454ba0c..4420bece0989fc1a627c2e775f9b923226018a39 100644 (file)
@@ -64,7 +64,7 @@ macro(curl_internal_test _curl_test)
   endif()
 endmacro()
 
-macro(optional_dependency _dependency)
+macro(curl_dependency_option _dependency)
   set(CURL_${_dependency} "AUTO" CACHE STRING "Build curl with ${_dependency} support (AUTO, ON or OFF)")
   set_property(CACHE CURL_${_dependency} PROPERTY STRINGS "AUTO" "ON" "OFF")
 
index 7dc1f99f022fd8626e87bafdaaa4f2813caf2b1a..773dad32ca00bb4e228e513830ff16daa4a8bf11 100644 (file)
@@ -35,7 +35,7 @@ include(CMakeFindDependencyMacro)
 if(@USE_OPENSSL@)
   find_dependency(OpenSSL @OPENSSL_VERSION_MAJOR@)
 endif()
-if(@USE_ZLIB@)
+if(@HAVE_LIBZ@)
   find_dependency(ZLIB @ZLIB_VERSION_MAJOR@)
 endif()
 
index aee20fe4381d4544f3b74b14a242db972cb0f72c..853554f9a848f964bbe81e446240d3821651cd58 100644 (file)
@@ -703,11 +703,9 @@ endif()
 # and before calling openssl_check_symbol_exists().
 
 set(HAVE_LIBZ OFF)
-set(USE_ZLIB OFF)
-optional_dependency(ZLIB)
+curl_dependency_option(ZLIB)
 if(ZLIB_FOUND)
   set(HAVE_LIBZ ON)
-  set(USE_ZLIB ON)
 
   # Depend on ZLIB via imported targets. This allows our dependents to
   # get our dependencies transitively.
@@ -2144,7 +2142,7 @@ if(NOT CURL_DISABLE_INSTALL)
   #   LIB_SELECTED
   #   TARGETS_EXPORT_NAME
   #   USE_OPENSSL
-  #   USE_ZLIB
+  #   HAVE_LIBZ
   configure_package_config_file("CMake/curl-config.cmake.in"
     "${_project_config}"
     INSTALL_DESTINATION ${CURL_INSTALL_CMAKE_DIR}