Fix lto optimization generating cmake errors

lagadic · Sep 24, 2024 · dff31fd · dff31fd
1 parent da11d4d
commit dff31fd
Showing 1 changed file with 7 additions and 8 deletions.
diff --git a/modules/python/bindings/CMakeLists.txt b/modules/python/bindings/CMakeLists.txt
@@ -49,27 +49,26 @@ file(MAKE_DIRECTORY "${bindings_gen_location}/src")
 get_target_property(PYTHON_CXX_FLAGS _visp COMPILE_OPTIONS)
 get_target_property( PYTHON_LINKER_FLAGS _visp LINK_OPTIONS)
 
-
-set_target_properties(_visp PROPERTIES COMPILE_FLAGS ${PYTHON_CXX_FLAGS})
 if(PYTHON_CXX_FLAGS STREQUAL "PYTHON_CXX_FLAGS-NOTFOUND")
   SET(PYTHON_CXX_FLAGS "") # Set to empty string
 else()
-  SET(PYTHON_CXX_FLAGS "${PYTHON_CXX_FLAGS} ") # A space to cleanly separate from existing content
+  SET(PYTHON_CXX_FLAGS "${PYTHON_CXX_FLAGS}")
 endif()
 if(PYTHON_LINKER_FLAGS STREQUAL "PYTHON_LINKER_FLAGS-NOTFOUND")
-SET(PYTHON_LINKER_FLAGS "") # Set to empty string
+  SET(PYTHON_LINKER_FLAGS "") # Set to empty string
 else()
-SET(PYTHON_LINKER_FLAGS "${PYTHON_LINKER_FLAGS} ") # A space to cleanly separate from existing content
+  SET(PYTHON_LINKER_FLAGS "${PYTHON_LINKER_FLAGS}")
 endif()
 CHECK_CXX_COMPILER_FLAG("-flto=auto" COMPILER_SUPPORTS_FLTOAUTO)
 if(COMPILER_SUPPORTS_FLTOAUTO)
   SET(PYTHON_CXX_FLAGS  "${PYTHON_CXX_FLAGS} -flto=auto")
   SET(PYTHON_LINKER_FLAGS  "${PYTHON_LINKER_FLAGS} -flto=auto")
 endif()
 
-set_target_properties(_visp PROPERTIES COMPILE_FLAGS ${PYTHON_CXX_FLAGS})
-set_target_properties(_visp PROPERTIES LINK_FLAGS ${PYTHON_LINKER_FLAGS})
-
+if(NOT PYTHON_CXX_FLAGS STREQUAL "")
+  set_target_properties(_visp PROPERTIES COMPILE_FLAGS ${PYTHON_CXX_FLAGS})
+  set_target_properties(_visp PROPERTIES LINK_FLAGS ${PYTHON_LINKER_FLAGS})
+endif()
 
 set_target_properties(_visp PROPERTIES
   LIBRARY_OUTPUT_DIRECTORY "${CMAKE_CURRENT_BINARY_DIR}"