romenskiy2012/curl
1
0
Fork 0
mirror of https://github.com/curl/curl.git synced 2026-05-30 10:17:28 +03:00
Code Issues Projects Releases Packages Wiki Activity

cmake: gather options recursively in curl_add_clang_tidy_test_target

Browse source 
Also look into `INTERFACE_INCLUDE_DIRECTORIES` target properties
for include directories.

Ref: #16973

Closes #17812
This commit is contained in:
Viktor Szakats 2025-07-04 05:25:27 +02:00
parent 5fb10b5476
commit b2c9e5ea10
No known key found for this signature in database
GPG key ID: B5ABD165E2AEF201
1 changed files with 21 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

29
CMake/Macros.cmake
View file

@ -96,16 +96,17 @@ macro(curl_prefill_type_size _type _size)
set(SIZEOF_${_type}_CODE "#define SIZEOF_${_type} ${_size}")
endmacro()
# Create a clang-tidy target for test targets
macro(curl_add_clang_tidy_test_target _target_clang_tidy _target)
if(CURL_CLANG_TIDY)
# Collect header directories and macro definitions from lib dependencies
set(_includes_l "")
set(_definitions_l "")
get_target_property(_libs ${_target} LINK_LIBRARIES)
# Internal: Recurse into target libraries and collect their include directories
# and macro definitions.
macro(curl_collect_target_options _target)
get_target_property(_libs ${_target} LINK_LIBRARIES)
if(_libs)
foreach(_lib IN LISTS _libs)
if(TARGET "${_lib}")
get_target_property(_val ${_lib} INTERFACE_INCLUDE_DIRECTORIES)
if(_val)
list(APPEND _includes_l ${_val})
endif()
get_target_property(_val ${_lib} INCLUDE_DIRECTORIES)
if(_val)
list(APPEND _includes_l ${_val})
@ -114,8 +115,20 @@ macro(curl_add_clang_tidy_test_target _target_clang_tidy _target)
if(_val)
list(APPEND _definitions_l ${_val})
endif()
curl_collect_target_options(${_lib})
endif()
endforeach()
endif()
endmacro()
# Create a clang-tidy target for test targets
macro(curl_add_clang_tidy_test_target _target_clang_tidy _target)
if(CURL_CLANG_TIDY)
# Collect header directories and macro definitions from lib dependencies
set(_includes_l "")
set(_definitions_l "")
curl_collect_target_options(${_target})
# Collect header directories applying to the target
get_directory_property(_includes_d INCLUDE_DIRECTORIES)