Skip to content

Commit

Permalink
build: do not support c-ares >= 1.33
Browse files Browse the repository at this point in the history
we don't support c-ares >= 1.33 yet. so error out when c-ares >= 1.33 is
detected.

Refs #2472

Signed-off-by: Kefu Chai <[email protected]>
  • Loading branch information
tchaikov authored and avikivity committed Oct 7, 2024
1 parent 3b2f6f7 commit 972258f
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
1 change: 1 addition & 0 deletions cmake/Findc-ares.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,7 @@ find_package_handle_standard_args (c-ares
if (c-ares_FOUND)
set (c-ares_LIBRARIES ${c-ares_LIBRARY})
set (c-ares_INCLUDE_DIRS ${c-ares_INCLUDE_DIR})
set (c-ares_VERSION ${PC_c-ares_VERSION})
if (NOT (TARGET c-ares::cares))
add_library (c-ares::cares UNKNOWN IMPORTED)

Expand Down
7 changes: 7 additions & 0 deletions cmake/SeastarDependencies.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -166,4 +166,11 @@ macro (seastar_find_dependencies)
else ()
find_package(Protobuf 2.5.0 REQUIRED)
endif ()

if (c-ares_VERSION VERSION_GREATER_EQUAL 1.33)
# https://github.com/scylladb/seastar/issues/2472
message (FATAL_ERROR
"c-ares ${c-ares_VERSION} is not supported. "
"Seastar requires c-ares version lower than 1.33")
endif ()
endmacro ()

0 comments on commit 972258f

Please sign in to comment.