From ed040f187375cea20b5cceab2a3697a5e305980b Mon Sep 17 00:00:00 2001 From: Sigurd Hellesvik Date: Wed, 6 Dec 2023 10:20:34 +0100 Subject: [PATCH] cmake: Show executable for memory report Firstly, COMMENT does not work for ninja. Therefore, change COMMENT to echo. With the addition of sysbuild, several memory reports can be printed for one build. Because of this, it is useful to know which executable each memory report is printed for, so adding full path to elf file. Signed-off-by: Sigurd Hellesvik --- CMakeLists.txt | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 3b0af4e34a3..be0dafe0fc1 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -1885,12 +1885,11 @@ endif() add_custom_command( TARGET ${logical_target_for_zephyr_elf} POST_BUILD + COMMAND ${CMAKE_COMMAND} -E echo "Generating files from ${PROJECT_BINARY_DIR}/${KERNEL_ELF_NAME} for board: ${BOARD}" ${post_build_commands} BYPRODUCTS ${post_build_byproducts} - COMMENT "Generating files from ${KERNEL_ELF_NAME} for board: ${BOARD}" COMMAND_EXPAND_LISTS - # NB: COMMENT only works for some CMake-Generators ) # To populate with hex files to merge, do the following: