Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resolve CMake Support for MCUboot Example #12

Merged
merged 6 commits into from
Sep 21, 2021
Merged

Resolve CMake Support for MCUboot Example #12

merged 6 commits into from
Sep 21, 2021

Conversation

sudotensor
Copy link
Owner

Please refer to the associated issue (#3) for more information. In summary, support for building the MCUboot example with the new mbed-tools was added; this involved the creation of CMakeLists.txt files and updating the library information for mcuboot.

Note: The FOTA examples now work on DISCO_L475VG_IOT01A; however, the build scripts haven't been updated just yet - this will follow soon.

Closes #3

Sudarshan Sreeram added 5 commits September 21, 2021 13:07
The mcuboot.lib files now point to lambda-shuttle/mcuboot, which
contains support for CMake.
The path now includes the full path to this header, which lives under
the flash_map_backend folder of the mbed-mcuboot port's include
directory.
@sudotensor sudotensor self-assigned this Sep 21, 2021

target_link_libraries(${APP_TARGET}
PUBLIC
mbed-os
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a reason to have different indentation ?

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh interesting... I could revert this to the previous style if you'd like. I maintained this style of indentation in the CMakeLists.txt files I wrote for both the mbed-mcuboot port and its examples. It's consistent with that followed in the mcuboot/boot/bootutil/CMakeLists.txt. I do notice that the indentation style of mcuboot/boot/espressif/CMakeLists.txt seems to match the previous style.

@sudotensor sudotensor merged commit 701246d into main Sep 21, 2021
@sudotensor sudotensor deleted the cmake-support branch September 21, 2021 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add CMakeLists.txt to MCUBoot example
2 participants