From 3918efd8e9275c849a5185f5d86b97c43f2bbd71 Mon Sep 17 00:00:00 2001 From: Emil Velikov Date: Sun, 14 Sep 2025 17:12:37 +0100 Subject: [PATCH] cmake: remove CMAKE_REQUIRED_* instances The tcl ones has not been required for over a decade since commit ffdba5b24 ("Remove check of Tcl_CreateNamespace in cmake build (not used any more) (bug #27119)"). While the top-level one, with the EXTRA_LIBS reshuffle/consolidation a few commits ago. Signed-off-by: Emil Velikov --- CMakeLists.txt | 1 - src/plugins/tcl/CMakeLists.txt | 2 -- 2 files changed, 3 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 67f43d123..ecbb3df47 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -313,7 +313,6 @@ endif() if(${CMAKE_SYSTEM_NAME} STREQUAL "FreeBSD") find_library(EXECINFO_LIB_PATH execinfo /usr/local/lib) - set(CMAKE_REQUIRED_LIBRARIES "${EXECINFO_LIB_PATH}") check_function_exists(backtrace HAVE_BACKTRACE) list(APPEND EXTRA_LIBS "execinfo") else() diff --git a/src/plugins/tcl/CMakeLists.txt b/src/plugins/tcl/CMakeLists.txt index c9bba6caf..d5932e207 100644 --- a/src/plugins/tcl/CMakeLists.txt +++ b/src/plugins/tcl/CMakeLists.txt @@ -28,8 +28,6 @@ add_library(tcl MODULE set_target_properties(tcl PROPERTIES PREFIX "") if(TCL_FOUND) - set(CMAKE_REQUIRED_INCLUDES "${TCL_INCLUDE_PATH}") - set(CMAKE_REQUIRED_LIBRARIES "${TCL_LIBRARY}") include_directories(${TCL_INCLUDE_PATH}) set(CMAKE_MODULE_LINKER_FLAGS "${CMAKE_MODULE_LINKER_FLAGS} ${TCL_LFLAGS}") target_link_libraries(tcl ${TCL_LIBRARY} weechat_plugins_scripts coverage_config)