REQUIRED_FLAGS should be given in C FLAGS not in library dependencies

182693ef · PRUVOST Florent · c869675f · 182693ef
Commit 182693ef authored Mar 22, 2018 by PRUVOST Florent
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 5 deletions

modules/find/FindSTARPU.cmake modules/find/FindSTARPU.cmake +2 -5

No files found.
--- a/modules/find/FindSTARPU.cmake
+++ b/modules/find/FindSTARPU.cmake
@@ -874,13 +874,10 @@ if( (NOT PKG_CONFIG_EXECUTABLE) OR (PKG_CONFIG_EXECUTABLE AND NOT STARPU_FOUND)

    if(STARPU_WORKS)
      # save link with dependencies
-      if (REQUIRED_FLAGS)
-        set(STARPU_LIBRARIES_DEP "${REQUIRED_FLAGS};${REQUIRED_LIBS}")
-      else()
-        set(STARPU_LIBRARIES_DEP "${REQUIRED_LIBS}")
-      endif()
+      set(STARPU_LIBRARIES_DEP "${REQUIRED_LIBS}")
      set(STARPU_LIBRARY_DIRS_DEP "${REQUIRED_LIBDIRS}")
      set(STARPU_INCLUDE_DIRS_DEP "${REQUIRED_INCDIRS}")
+      set(STARPU_C_FLAGS "${REQUIRED_FLAGS}")
      set(STARPU_LINKER_FLAGS "${REQUIRED_LDFLAGS}")
      list(REMOVE_DUPLICATES STARPU_LIBRARY_DIRS_DEP)
      list(REMOVE_DUPLICATES STARPU_INCLUDE_DIRS_DEP)