diff --git a/components/esp_partition/host_test/partition_api_test/CMakeLists.txt b/components/esp_partition/host_test/partition_api_test/CMakeLists.txt index 3986bdb3140..469a614a143 100644 --- a/components/esp_partition/host_test/partition_api_test/CMakeLists.txt +++ b/components/esp_partition/host_test/partition_api_test/CMakeLists.txt @@ -7,5 +7,3 @@ set(COMPONENTS main) list(APPEND EXTRA_COMPONENT_DIRS "$ENV{IDF_PATH}/tools/mocks/freertos/") project(partition_api_test) - -add_dependencies(partition_api_test.elf partition-table) diff --git a/components/esp_partition/host_test/partition_api_test/README.md b/components/esp_partition/host_test/partition_api_test/README.md index 8e2e38e9204..10b893d003f 100644 --- a/components/esp_partition/host_test/partition_api_test/README.md +++ b/components/esp_partition/host_test/partition_api_test/README.md @@ -8,11 +8,10 @@ Source the IDF environment as usual. Once this is done, build the application: ```bash -idf.py build partition-table +idf.py build ``` -Note that for the time being, `partition-table` target needs to be built manually. # Run ```bash -`build/partition_api_test.elf` +idf.py monitor ``` diff --git a/components/esp_rom/host_test/rom_test/README.md b/components/esp_rom/host_test/rom_test/README.md index 4e99101c90f..22e67c9eae3 100644 --- a/components/esp_rom/host_test/rom_test/README.md +++ b/components/esp_rom/host_test/rom_test/README.md @@ -19,10 +19,8 @@ First, make sure that the target is set to Linux. Run `idf.py --preview set-targ ## Run -IDF monitor doesn't work yet for Linux. You have to run the app manually: - ```bash -./build/test_rom_host.elf +idf.py monitor ``` ## Example Output @@ -30,7 +28,7 @@ IDF monitor doesn't work yet for Linux. You have to run the app manually: Ideally, all tests pass, which is indicated by "All tests passed" in the last line: ```bash -$ ./build/test_rom_host.elf +$ idf.py monitor test =============================================================================== All tests passed (8 assertions in 6 test cases) diff --git a/components/log/host_test/log_test/README.md b/components/log/host_test/log_test/README.md index 4c37b52c7dc..47d25b47165 100644 --- a/components/log/host_test/log_test/README.md +++ b/components/log/host_test/log_test/README.md @@ -19,10 +19,8 @@ First, make sure that the target is set to Linux. Run `idf.py --preview set-targ ## Run -IDF monitor doesn't work yet for Linux. You have to run the app manually: - ```bash -./build/test_log_host.elf +idf.py monitor ``` ## Example Output @@ -30,7 +28,7 @@ IDF monitor doesn't work yet for Linux. You have to run the app manually: Ideally, all tests pass, which is indicated by "All tests passed" in the last line: ```bash -$ ./build/test_log_host.elf +$ idf.py monitor =============================================================================== All tests passed (8 assertions in 6 test cases) ``` diff --git a/components/nvs_flash/host_test/nvs_host_test/CMakeLists.txt b/components/nvs_flash/host_test/nvs_host_test/CMakeLists.txt index 655d83dcf0a..cd3f3ebece4 100644 --- a/components/nvs_flash/host_test/nvs_host_test/CMakeLists.txt +++ b/components/nvs_flash/host_test/nvs_host_test/CMakeLists.txt @@ -8,5 +8,3 @@ set(COMPONENTS main) list(APPEND EXTRA_COMPONENT_DIRS "$ENV{IDF_PATH}/tools/mocks/freertos/") project(nvs_host_test) - -add_dependencies(nvs_host_test.elf partition-table) diff --git a/components/nvs_flash/host_test/nvs_page_test/README.md b/components/nvs_flash/host_test/nvs_page_test/README.md index cf48e50a710..65699e802ff 100644 --- a/components/nvs_flash/host_test/nvs_page_test/README.md +++ b/components/nvs_flash/host_test/nvs_page_test/README.md @@ -33,10 +33,8 @@ First, make sure that the target is set to Linux. Run `idf.py --preview set-targ ## Run -IDF monitor doesn't work yet for Linux. You have to run the app manually: - ```bash -./build/host_nvs_page_test.elf +idf.py monitor ``` ## Coverage @@ -48,7 +46,7 @@ To generate the coverage, run: `idf.py coverage`. Afterwards, you can view the c Ideally, all tests pass, which is indicated by the last two log lines after the dashed line: ```bash -build/host_nvs_page_test.elf +$ idf.py monitor ../main/nvs_page_test.cpp:880:test_Page_load_reading_header_fails:PASS ../main/nvs_page_test.cpp:881:test_Page_load_reading_data_fails:PASS ../main/nvs_page_test.cpp:882:test_Page_load__uninitialized_page_has_0xfe:PASS diff --git a/components/partition_table/CMakeLists.txt b/components/partition_table/CMakeLists.txt index 31260c1f125..f386028612b 100644 --- a/components/partition_table/CMakeLists.txt +++ b/components/partition_table/CMakeLists.txt @@ -96,6 +96,24 @@ else() "Either change partition table in menuconfig or create this input file.") endif() +if(${target} STREQUAL "linux" AND EXISTS ${partition_csv}) + # partition-table target is normally invoked as a dependency of 'flash' target. + # However, when building for "linux" target, 'flash' target doesn't exist, + # so we need to attach the partition table build to the executable target. + # + # The problem is that the executable target is not yet defined + # when the component CMakeLists.txt file is evaluated, so we + # can only get it as a generator expression. But generator expressions + # can't be used in 'add_dependencies': + # https://gitlab.kitware.com/cmake/cmake/-/issues/19467 + # + # Therefore attach partition-table to the internal __idf_build_target + # target. This is a hack, since that target name is an implementation detail + # of the build system. + + add_dependencies(__idf_build_target partition-table) +endif() + # Add signing steps if(CONFIG_SECURE_SIGNED_APPS_ECDSA_SCHEME) if(CONFIG_SECURE_BOOT_BUILD_SIGNED_BINARIES) diff --git a/components/spiffs/host_test/CMakeLists.txt b/components/spiffs/host_test/CMakeLists.txt index 03f7125aedf..81a2485a617 100644 --- a/components/spiffs/host_test/CMakeLists.txt +++ b/components/spiffs/host_test/CMakeLists.txt @@ -22,4 +22,5 @@ set_property( DIRECTORY APPEND PROPERTY ADDITIONAL_CLEAN_FILES "../image.bin") -add_dependencies(host_test_spiffs.elf partition-table image.bin) + +add_dependencies(host_test_spiffs.elf image.bin) diff --git a/components/spiffs/host_test/README.md b/components/spiffs/host_test/README.md index dc6f6df5fd2..a545955fa39 100644 --- a/components/spiffs/host_test/README.md +++ b/components/spiffs/host_test/README.md @@ -13,5 +13,5 @@ idf.py build # Run ```bash -build/host_test_spiffs.elf +idf.py monitor ``` diff --git a/tools/idf_monitor_base/pc_address_matcher.py b/tools/idf_monitor_base/pc_address_matcher.py index 9a97e294839..c0c69dc9117 100644 --- a/tools/idf_monitor_base/pc_address_matcher.py +++ b/tools/idf_monitor_base/pc_address_matcher.py @@ -7,8 +7,8 @@ from elftools.elf.elffile import ELFFile class PcAddressMatcher(object): """ - Class for detecting potentional addresses which will consequently run through the external addr2line command to - indentify and print information about it. + Class for detecting potential addresses which will consequently run through the external addr2line command to + identify and print information about it. The input to this class is the path to the ELF file. Addresses of sections with executable flag are stored and used later for lookup. @@ -18,6 +18,14 @@ class PcAddressMatcher(object): self.intervals = [] try: with open(elf_path, 'rb') as f: + # Is this an ELF file? + elf_magic = f.read(4) + if elf_magic != b'\x7fELF': + # Probably not an ELF file + # (could be Mach-O format on macOS, for example) + raise NotImplementedError() + f.seek(0) + elf = ELFFile(f) for section in elf.iter_sections(): @@ -30,13 +38,15 @@ class PcAddressMatcher(object): except FileNotFoundError: # ELF file is just an optional argument pass + except NotImplementedError: + pass # sort them in order to have faster lookup self.intervals = sorted(self.intervals) def is_executable_address(self, addr): # type: (int) -> bool """ - Returns True/False depending on of "addr" is in one of the ELF sections with executable flag set. + Returns True/False if "addr" is in one of the ELF sections with executable flag set. """ for start, end in self.intervals: