Skip to content

Conversation

@nordicjm
Copy link
Collaborator

This sample is very outdated and there has been a sysbuild MCUboot sample in zephyr for a number of years now

This sample is very outdated and there has been a sysbuild MCUboot
sample in zephyr for a number of years now

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
Copy link
Member

@d3zd3z d3zd3z left a comment

Choose a reason for hiding this comment

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

So, this used to be how I tested mcuboot upgrades. I guess the real question is, does any of this directory belong? I'm not sure I see the point in removing just the hello-world part. Or should the samples/zephyr scripts and such just be configured to build something out of the Zephyr tree?

@nandojve
Copy link
Contributor

Hi @d3zd3z ,

So, this used to be how I tested mcuboot upgrades. I guess the real question is, does any of this directory belong? I'm not sure I see the point in removing just the hello-world part. Or should the samples/zephyr scripts and such just be configured to build something out of the Zephyr tree?

Did you look at #2589 ?
That is everything that I need to make sure twister works.

@nordicjm
Copy link
Collaborator Author

So, this used to be how I tested mcuboot upgrades. I guess the real question is, does any of this directory belong? I'm not sure I see the point in removing just the hello-world part. Or should the samples/zephyr scripts and such just be configured to build something out of the Zephyr tree?

There is https://github.com/zephyrproject-rtos/zephyr/tree/main/samples/sysbuild/with_mcuboot which includes an application and MCUboot and is tested in zephyr's CI

Copy link
Member

@d3zd3z d3zd3z left a comment

Choose a reason for hiding this comment

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

This is indeed dated, and redundant with tests already in Zephyr.

@nordicjm nordicjm merged commit 1027003 into mcu-tools:main Jan 23, 2026
64 checks passed
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.

4 participants