cmake: set BUILDING_LIBCURL directly for unit test targets

To avoid adding this macro to the global `CURL_DEBUG_MACROS` variable,
which may be used for targets defined after unit tests, and where this
macro may not be necessary.

As of this commit unit tests are defined last, so extending the global
variable did not cause any issue.

Follow-up to 220eda34cd #17259

Closes #17264
This commit is contained in:
Viktor Szakats 2025-05-07 01:29:32 +02:00
parent e3802ab489
commit c6d0524265
No known key found for this signature in database
GPG key ID: B5ABD165E2AEF201

View file

@ -39,9 +39,6 @@ if(CURL_TEST_BUNDLES)
set(units_SOURCES "unit_bundle.c")
endif()
# unit tests are small pretend-libcurl-programs
list(APPEND CURL_DEBUG_MACROS "BUILDING_LIBCURL")
foreach(_target IN LISTS UNITPROGS)
set(_target_name "${_target}")
add_executable(${_target_name} EXCLUDE_FROM_ALL ${${_target}_SOURCES})
@ -54,6 +51,8 @@ foreach(_target IN LISTS UNITPROGS)
"${PROJECT_SOURCE_DIR}/tests/libtest"
)
set_property(TARGET ${_target_name} APPEND PROPERTY COMPILE_DEFINITIONS "${CURL_DEBUG_MACROS}")
# unit tests are small pretend-libcurl-programs
set_property(TARGET ${_target_name} APPEND PROPERTY COMPILE_DEFINITIONS "BUILDING_LIBCURL")
if(CURL_TEST_BUNDLES)
target_include_directories(${_target_name} PRIVATE "${PROJECT_SOURCE_DIR}/tests/unit")
endif()