From 5ad118d7d227af11ea0396b15c6e9ea8b257fd49 Mon Sep 17 00:00:00 2001 From: Tom Finegan Date: Thu, 4 Feb 2021 15:35:12 -0800 Subject: [PATCH] draco: Support multiple link flags in older CMakes Stringify the flag list and apply the flags in CMakes before v3.13. --- cmake/draco_targets.cmake | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/cmake/draco_targets.cmake b/cmake/draco_targets.cmake index 30f8501..a639049 100644 --- a/cmake/draco_targets.cmake +++ b/cmake/draco_targets.cmake @@ -110,10 +110,11 @@ macro(draco_add_executable) if(exe_LINK_FLAGS OR DRACO_EXE_LINKER_FLAGS) if(${CMAKE_VERSION} VERSION_LESS "3.13") - set(link_flags ${exe_LINK_FLAGS} ${DRACO_EXE_LINKER_FLAGS}) + list(APPEND exe_LINK_FLAGS "${DRACO_EXE_LINKER_FLAGS}") + # LINK_FLAGS is managed as a string. + draco_set_and_stringify(SOURCE "${exe_LINK_FLAGS}" DEST exe_LINK_FLAGS) set_target_properties(${exe_NAME} - PROPERTIES LINK_FLAGS ${exe_LINK_FLAGS} - ${DRACO_EXE_LINKER_FLAGS}) + PROPERTIES LINK_FLAGS "${exe_LINK_FLAGS}") else() target_link_options(${exe_NAME} PRIVATE ${exe_LINK_FLAGS} ${DRACO_EXE_LINKER_FLAGS})