Rework dependencies about custom target to be more CMake compliant
authorRomain Forlot <romain.forlot@iot.bzh>
Tue, 9 May 2017 11:17:16 +0000 (13:17 +0200)
committerRomain Forlot <romain.forlot@iot.bzh>
Tue, 9 May 2017 11:17:16 +0000 (13:17 +0200)
CMake dependencies about custom target is to added with add_dependencies()
DEPENDS options is more for output file dependencies.

Change-Id: I3fb6b17e0de97bbb495471d50562d41f6ab549f0
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
reference/etc/macros.cmake

index b89094a..210ed91 100644 (file)
@@ -243,13 +243,14 @@ if(EXTRA_DEPENDENCIES_ORDER)
        )
 endif()
 
-# Print developer helper message when everything is done
+# Print developer helper message when build is done
 # -------------------------------------------------------
 macro(project_closing_msg)
        if(CLOSING_MESSAGE AND GLOBAL_TARGET_LIST)
-               add_custom_target(${PROJECT_NAME}_done ALL
-                       DEPENDS ${DEPENDENCIES_TARGET} ${GLOBAL_TARGET_LIST}
+               add_custom_target(${PROJECT_NAME}_build_done ALL
                        COMMAND ${CMAKE_COMMAND} -E cmake_echo_color --cyan "++ ${CLOSING_MESSAGE}"
                )
+                add_dependencies(${PROJECT_NAME}_build_done
+                       ${DEPENDENCIES_TARGET} ${GLOBAL_TARGET_LIST})
        endif()
 endmacro()