Skip to content

Commit 67ae4b5

Browse files
authored
Add missing file type for esp32 binary. (aws#1550)
* Applied Espressif elf suffix patch.
1 parent ae3a0b7 commit 67ae4b5

File tree

6 files changed

+24
-12
lines changed

6 files changed

+24
-12
lines changed

vendors/espressif/boards/esp32/CMakeLists.txt

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
set(CMAKE_EXECUTABLE_SUFFIX ".elf")
2+
13
# -------------------------------------------------------------------------------------------------
24
# Amazon FreeRTOS Console metadata
35
# -------------------------------------------------------------------------------------------------
@@ -63,6 +65,8 @@ target_link_options(
6365
INTERFACE ${linker_flags}
6466
)
6567

68+
set(CMAKE_EXECUTABLE_SUFFIX ".elf")
69+
6670
# -------------------------------------------------------------------------------------------------
6771
# Amazon FreeRTOS portable layers
6872
# -------------------------------------------------------------------------------------------------

vendors/espressif/esp-idf/components/bootloader/subproject/CMakeLists.txt

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,21 +19,21 @@ set(COMPONENT_REQUIRES_COMMON log esp32 soc)
1919
include("${IDF_PATH}/tools/cmake/project.cmake")
2020
project(bootloader)
2121

22-
target_linker_script(bootloader.elf
22+
target_linker_script(bootloader
2323
"main/esp32.bootloader.ld"
2424
"main/esp32.bootloader.rom.ld"
2525
)
2626

2727
# as cmake won't attach linker args to a header-only library, attach
28-
# linker args directly to the bootloader.elf
28+
# linker args directly to the bootloader
2929
set(ESP32_BOOTLOADER_LINKER_SCRIPTS
3030
"${IDF_PATH}/components/esp32/ld/esp32.rom.ld"
3131
"${IDF_PATH}/components/esp32/ld/esp32.rom.spiram_incompatible_fns.ld"
3232
"${IDF_PATH}/components/esp32/ld/esp32.peripherals.ld")
3333

34-
target_linker_script(bootloader.elf ${ESP32_BOOTLOADER_LINKER_SCRIPTS})
34+
target_linker_script(bootloader ${ESP32_BOOTLOADER_LINKER_SCRIPTS})
3535

36-
target_link_libraries(bootloader.elf gcc)
36+
target_link_libraries(bootloader gcc)
3737

3838
set(secure_boot_signing_key ${SECURE_BOOT_SIGNING_KEY})
3939

vendors/espressif/esp-idf/components/esptool_py/project_include.cmake

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ endif()
8484
#
8585
add_custom_command(OUTPUT "${IDF_BUILD_ARTIFACTS_DIR}/${unsigned_project_binary}"
8686
COMMAND ${ESPTOOLPY} elf2image ${ESPTOOLPY_ELF2IMAGE_FLASH_OPTIONS} ${ESPTOOLPY_ELF2IMAGE_OPTIONS}
87-
-o "${IDF_BUILD_ARTIFACTS_DIR}/${unsigned_project_binary}" "${IDF_PROJECT_EXECUTABLE}"
87+
-o "${IDF_BUILD_ARTIFACTS_DIR}/${unsigned_project_binary}" "$<TARGET_FILE:${IDF_PROJECT_EXECUTABLE}>"
8888
DEPENDS ${IDF_PROJECT_EXECUTABLE}
8989
VERBATIM
9090
WORKING_DIRECTORY ${CMAKE_BINARY_DIR}
@@ -106,11 +106,7 @@ if(NOT BOOTLOADER_BUILD AND
106106
)
107107
endif()
108108

109-
if(NOT BOOTLOADER_BUILD)
110-
add_custom_target(app ALL DEPENDS "${IDF_BUILD_ARTIFACTS_DIR}/${IDF_PROJECT_BIN}")
111-
else()
112-
add_custom_target(bootloader ALL DEPENDS "${IDF_BUILD_ARTIFACTS_DIR}/${IDF_PROJECT_BIN}")
113-
endif()
109+
add_custom_target(app ALL DEPENDS "${IDF_BUILD_ARTIFACTS_DIR}/${IDF_PROJECT_BIN}")
114110

115111
if(NOT BOOTLOADER_BUILD AND
116112
CONFIG_SECURE_BOOT_ENABLED AND

vendors/espressif/esp-idf/tools/cmake/idf_functions.cmake

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -408,8 +408,18 @@ macro(idf_import_components var idf_path build_path)
408408
# Write project description JSON file
409409
make_json_list("${BUILD_COMPONENTS}" build_components_json)
410410
make_json_list("${BUILD_COMPONENT_PATHS}" build_component_paths_json)
411+
411412
configure_file("${IDF_PATH}/tools/cmake/project_description.json.in"
413+
"${IDF_BUILD_ARTIFACTS_DIR}/project_description.json.in")
414+
415+
file(GENERATE OUTPUT "${IDF_BUILD_ARTIFACTS_DIR}/project_description.json"
416+
INPUT "${IDF_BUILD_ARTIFACTS_DIR}/project_description.json.in")
417+
418+
set_property(DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}" APPEND PROPERTY
419+
ADDITIONAL_MAKE_CLEAN_FILES
420+
"${IDF_BUILD_ARTIFACTS_DIR}/project_description.json.in"
412421
"${IDF_BUILD_ARTIFACTS_DIR}/project_description.json")
422+
413423
unset(build_components_json)
414424
unset(build_component_paths_json)
415425
endif()

vendors/espressif/esp-idf/tools/cmake/project.cmake

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,8 +94,10 @@ macro(project name)
9494

9595
__project(${name} C CXX ASM)
9696

97+
set(CMAKE_EXECUTABLE_SUFFIX ".elf")
98+
9799
set(IDF_BUILD_ARTIFACTS ON)
98-
set(IDF_PROJECT_EXECUTABLE ${CMAKE_PROJECT_NAME}.elf)
100+
set(IDF_PROJECT_EXECUTABLE ${CMAKE_PROJECT_NAME})
99101
set(IDF_BUILD_ARTIFACTS_DIR ${CMAKE_BINARY_DIR})
100102

101103
if(MAIN_SRCS)

vendors/espressif/esp-idf/tools/cmake/project_description.json.in

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
"build_dir": "${IDF_BUILD_ARTIFACTS_DIR}",
55
"config_file": "${SDKCONFIG}",
66
"config_defaults": "${IDF_SDKCONFIG_DEFAULTS}",
7-
"app_elf": "${IDF_PROJECT_EXECUTABLE}",
7+
"app_elf": "$<TARGET_FILE_NAME:${IDF_PROJECT_EXECUTABLE}>",
88
"app_bin": "${IDF_PROJECT_BIN}",
99
"git_revision": "${IDF_VER}",
1010
"phy_data_partition": "${CONFIG_ESP32_PHY_INIT_DATA_IN_PARTITION}",

0 commit comments

Comments
 (0)