From 0b5260366779ee781a9af1057d340582476e0f6c Mon Sep 17 00:00:00 2001 From: Peter Harper <77111776+peterharperuk@users.noreply.github.com> Date: Tue, 6 Dec 2022 23:45:57 +0000 Subject: [PATCH] Fix build issues with spaces and parentheses (#1136) Add VERBATIM to add_custom_command Fixes #1043 --- src/rp2_common.cmake | 12 +++++++----- src/rp2_common/boot_stage2/CMakeLists.txt | 5 +++-- src/rp2_common/cyw43_driver/CMakeLists.txt | 2 +- tools/CMakeLists.txt | 5 +++-- 4 files changed, 14 insertions(+), 10 deletions(-) diff --git a/src/rp2_common.cmake b/src/rp2_common.cmake index bc82e7c..7ad6e64 100644 --- a/src/rp2_common.cmake +++ b/src/rp2_common.cmake @@ -3,18 +3,20 @@ enable_language(ASM) function(pico_add_hex_output TARGET) - add_custom_command(TARGET ${TARGET} POST_BUILD COMMAND ${CMAKE_OBJCOPY} -Oihex $ $>,$,$>.hex) + add_custom_command(TARGET ${TARGET} POST_BUILD COMMAND ${CMAKE_OBJCOPY} -Oihex $ $>,$,$>.hex + VERBATIM) endfunction() function(pico_add_bin_output TARGET) - add_custom_command(TARGET ${TARGET} POST_BUILD COMMAND ${CMAKE_OBJCOPY} -Obinary $ $>,$,$>.bin) + add_custom_command(TARGET ${TARGET} POST_BUILD COMMAND ${CMAKE_OBJCOPY} -Obinary $ $>,$,$>.bin + VERBATIM) endfunction() function(pico_add_dis_output TARGET) add_custom_command(TARGET ${TARGET} POST_BUILD - COMMAND ${CMAKE_OBJDUMP} -h $ >$>,$,$>.dis - COMMAND ${CMAKE_OBJDUMP} -d $ >>$>,$,$>.dis - ) + COMMAND ${CMAKE_OBJDUMP} -h $ > $>,$,$>.dis + COMMAND ${CMAKE_OBJDUMP} -d $ >> $>,$,$>.dis + VERBATIM) endfunction() function(pico_add_extra_outputs TARGET) diff --git a/src/rp2_common/boot_stage2/CMakeLists.txt b/src/rp2_common/boot_stage2/CMakeLists.txt index a2f3960..a130f41 100644 --- a/src/rp2_common/boot_stage2/CMakeLists.txt +++ b/src/rp2_common/boot_stage2/CMakeLists.txt @@ -63,12 +63,13 @@ function(pico_define_boot_stage2 NAME SOURCES) find_package (Python3 REQUIRED COMPONENTS Interpreter) add_custom_target(${NAME}_bin DEPENDS ${ORIGINAL_BIN}) - add_custom_command(OUTPUT ${ORIGINAL_BIN} DEPENDS ${NAME} COMMAND ${CMAKE_OBJCOPY} -Obinary $ ${ORIGINAL_BIN}) + add_custom_command(OUTPUT ${ORIGINAL_BIN} DEPENDS ${NAME} COMMAND ${CMAKE_OBJCOPY} -Obinary $ ${ORIGINAL_BIN} + VERBATIM) add_custom_target(${NAME}_padded_checksummed_asm DEPENDS ${PADDED_CHECKSUMMED_ASM}) add_custom_command(OUTPUT ${PADDED_CHECKSUMMED_ASM} DEPENDS ${ORIGINAL_BIN} COMMAND ${Python3_EXECUTABLE} ${PICO_BOOT_STAGE2_DIR}/pad_checksum -s 0xffffffff ${ORIGINAL_BIN} ${PADDED_CHECKSUMMED_ASM} - ) + VERBATIM) add_library(${NAME}_library INTERFACE) add_dependencies(${NAME}_library ${NAME}_padded_checksummed_asm) diff --git a/src/rp2_common/cyw43_driver/CMakeLists.txt b/src/rp2_common/cyw43_driver/CMakeLists.txt index 8951df4..42e2a30 100644 --- a/src/rp2_common/cyw43_driver/CMakeLists.txt +++ b/src/rp2_common/cyw43_driver/CMakeLists.txt @@ -58,7 +58,7 @@ if (EXISTS ${PICO_CYW43_DRIVER_PATH}/${CYW43_DRIVER_TEST_FILE}) --redefine-sym _binary_${CYW43_FIRMWARE_BIN_}_end=${CYW43_FIRMWARE_PRETTY}_end --redefine-sym _binary_${CYW43_FIRMWARE_BIN_}_size=${CYW43_FIRMWARE_PRETTY}_size ${CYW43_FIRMWARE_BIN} ${CYW43_FIRMWARE_OBJ} - ) + VERBATIM) add_library(cyw43_driver_picow INTERFACE) target_sources(cyw43_driver_picow INTERFACE diff --git a/tools/CMakeLists.txt b/tools/CMakeLists.txt index 3ae3b0a..7433eaa 100644 --- a/tools/CMakeLists.txt +++ b/tools/CMakeLists.txt @@ -33,7 +33,7 @@ function(pico_generate_pio_header TARGET PIO) add_custom_command(OUTPUT ${HEADER} DEPENDS ${PIO} COMMAND Pioasm -o ${OUTPUT_FORMAT} ${PIO} ${HEADER} - ) + VERBATIM) add_dependencies(${TARGET} ${HEADER_GEN_TARGET}) get_target_property(target_type ${TARGET} TYPE) if ("INTERFACE_LIBRARY" STREQUAL "${target_type}") @@ -59,6 +59,7 @@ function(pico_add_uf2_output TARGET) endif() if (ELF2UF2_FOUND) add_custom_command(TARGET ${TARGET} POST_BUILD - COMMAND ELF2UF2 $ ${output_path}$>,$,$>.uf2) + COMMAND ELF2UF2 $ ${output_path}$>,$,$>.uf2 + VERBATIM) endif() endfunction()