]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
cmake: s/c-ares::c-ares/c-ares::cares/ 48987/head
authorKefu Chai <tchaikov@gmail.com>
Mon, 21 Nov 2022 16:44:31 +0000 (00:44 +0800)
committerKefu Chai <tchaikov@gmail.com>
Mon, 21 Nov 2022 17:17:30 +0000 (01:17 +0800)
to be aligned with the c-ares upstream, which provides
c-ares-config.cmake in c-ares v1.17 and up. this cmake config file
is also packaged by fedora and CentOS stream in c-ares-devel-1.17.1 and
up.

this change prepares us for bumping up Seastar submodule, which also
has a similar change: https://github.com/scylladb/seastar/commit/aea45c98f255e2013d4b9b399944e9810ee1d972

Signed-off-by: Kefu Chai <tchaikov@gmail.com>
cmake/modules/Buildc-ares.cmake
cmake/modules/Findc-ares.cmake

index ff9452095fe4f0410ca81055fc6414e13ad23d51..022f2cd32619320a321715b0ed6bfdaa8281b50c 100644 (file)
@@ -11,12 +11,14 @@ function(build_c_ares)
     BINARY_DIR "${C-ARES_BINARY_DIR}"
     BUILD_COMMAND ${CMAKE_COMMAND} --build <BINARY_DIR>
     INSTALL_COMMAND "")
-  add_library(c-ares::c-ares STATIC IMPORTED)
-  add_dependencies(c-ares::c-ares c-ares_ext)
-  set_target_properties(c-ares::c-ares PROPERTIES
+  add_library(c-ares::cares STATIC IMPORTED)
+  add_dependencies(c-ares::cares c-ares_ext)
+  set_target_properties(c-ares::cares PROPERTIES
     INTERFACE_INCLUDE_DIRECTORIES "${C-ARES_SOURCE_DIR};${C-ARES_BINARY_DIR}"
     IMPORTED_LINK_INTERFACE_LANGUAGES "C"
     IMPORTED_LOCATION "${C-ARES_BINARY_DIR}/lib/libcares.a")
   # to appease find_package()
-  add_custom_target(c-ares DEPENDS c-ares::c-ares)
+  add_custom_target(c-ares DEPENDS c-ares::cares)
+  # to be compatible with old Seastar
+  add_library(c-ares::c-ares ALIAS c-ares::cares)
 endfunction()
index 6386f9acecb9788d6018661e33484ac1994c3f00..1e26c5c1c0fcadc9e746af7fda1d3c419e321835 100644 (file)
@@ -21,10 +21,12 @@ find_package_handle_standard_args(c-ares
     c-ares_LIBRARY
   VERSION_VAR c-ares_VERSION)
 
-if(c-ares_FOUND AND NOT (TARGET c-ares::c-ares))
-  add_library(c-ares::c-ares UNKNOWN IMPORTED)
-  set_target_properties(c-ares::c-ares PROPERTIES
+if(c-ares_FOUND AND NOT (TARGET c-ares::cares))
+  add_library(c-ares::cares UNKNOWN IMPORTED)
+  set_target_properties(c-ares::cares PROPERTIES
     INTERFACE_INCLUDE_DIRECTORIES "${c-ares_INCLUDE_DIR}"
     IMPORTED_LINK_INTERFACE_LANGUAGES "C"
     IMPORTED_LOCATION "${c-ares_LIBRARY}")
+  # to be compatible with old Seastar
+  add_library(c-ares::c-ares ALIAS c-ares::cares)
 endif()