mirror of
https://github.com/Mbed-TLS/mbedtls.git
synced 2025-07-28 00:21:48 +03:00
cmake: Fix added sources to targets
In cmake version < 3.0, the SOURCES property on targets cannot be
modified after the target is defined. There are several instances in
the code that were using `target_properties()`, which is not available
in the older versions of cmake. Unfortunately, the workaround in #3801
(381c1078fc
) assumes that this SOURCES property can be modified.
Work around this by building up any necessary sources before declaring
the target. This is more awkward, but needed to continue to be able to
support the old versions of cmake.
Fixes #3788.
Signed-off-by: David Brown <david.brown@linaro.org>
This commit is contained in:
@ -32,20 +32,24 @@ set(executables_with_common_c
|
||||
|
||||
foreach(exe IN LISTS executables_no_common_c executables_with_common_c)
|
||||
|
||||
add_executable(${exe} ${exe}.c $<TARGET_OBJECTS:mbedtls_test>)
|
||||
set(exe_sources ${exe}.c $<TARGET_OBJECTS:mbedtls_test>)
|
||||
if(NOT FUZZINGENGINE_LIB)
|
||||
list(APPEND exe_sources onefile.c)
|
||||
endif()
|
||||
|
||||
# This emulates "if ( ... IN_LIST ... )" which becomes available in CMake 3.3
|
||||
list(FIND executables_with_common_c ${exe} exe_index)
|
||||
if(${exe_index} GREATER -1)
|
||||
list(APPEND exe_sources common.c)
|
||||
endif()
|
||||
|
||||
add_executable(${exe} ${exe_sources})
|
||||
|
||||
if (NOT FUZZINGENGINE_LIB)
|
||||
target_link_libraries(${exe} ${libs})
|
||||
set_property(TARGET ${exe} APPEND PROPERTY SOURCES onefile.c)
|
||||
else()
|
||||
target_link_libraries(${exe} ${libs} FuzzingEngine)
|
||||
SET_TARGET_PROPERTIES(${exe} PROPERTIES LINKER_LANGUAGE CXX)
|
||||
endif()
|
||||
|
||||
# This emulates "if ( ... IN_LIST ... )" which becomes available in CMake 3.3
|
||||
list(FIND executables_with_common_c ${exe} exe_index)
|
||||
if (${exe_index} GREATER -1)
|
||||
set_property(TARGET ${exe} APPEND PROPERTY SOURCES common.c)
|
||||
endif()
|
||||
|
||||
endforeach()
|
||||
|
Reference in New Issue
Block a user