Use set() instead of list(APPEND to make a list

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
ros2 · Mar 16, 2022 · 2fe9d3a · 2fe9d3a
1 parent 0f59e02
commit 2fe9d3a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/rosidl_typesupport_fastrtps_c/cmake/rosidl_typesupport_fastrtps_c_generate_interfaces.cmake b/rosidl_typesupport_fastrtps_c/cmake/rosidl_typesupport_fastrtps_c_generate_interfaces.cmake
@@ -122,9 +122,9 @@ target_compile_definitions(${rosidl_generate_interfaces_TARGET}${_target_suffix}
   PRIVATE "ROSIDL_TYPESUPPORT_FASTRTPS_C_BUILDING_DLL_${PROJECT_NAME}")
 
 if(NOT WIN32)
-  list(APPEND _target_compile_flags -Wall -Wextra -Wpedantic)
+  set(_target_compile_flags -Wall -Wextra -Wpedantic)
 else()
-  list(APPEND _target_compile_flags /W4)
+  set(_target_compile_flags /W4)
 endif()
 target_compile_options(${rosidl_generate_interfaces_TARGET}${_target_suffix} PRIVATE ${_target_compile_flags})
 

diff --git a/..._typesupport_fastrtps_cpp/cmake/rosidl_typesupport_fastrtps_cpp_generate_interfaces.cmake b/..._typesupport_fastrtps_cpp/cmake/rosidl_typesupport_fastrtps_cpp_generate_interfaces.cmake
@@ -132,9 +132,9 @@ target_compile_definitions(${rosidl_generate_interfaces_TARGET}${_target_suffix}
 
 # Set compiler flags
 if(NOT WIN32)
-  list(APPEND _target_compile_flags -Wall -Wextra -Wpedantic)
+  set(_target_compile_flags -Wall -Wextra -Wpedantic)
 else()
-  list(APPEND _target_compile_flags /W4)
+  set(_target_compile_flags /W4)
 endif()
 target_compile_options(${rosidl_generate_interfaces_TARGET}${_target_suffix} PRIVATE ${_target_compile_flags})