-
Notifications
You must be signed in to change notification settings - Fork 683
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
Revert RAM load single application commits #2106
Conversation
This reverts commit 4893193. Signed-off-by: Jamie McCrae <[email protected]>
This reverts commit 249a71d. Signed-off-by: Jamie McCrae <[email protected]>
…ode" This reverts commit 5c67fb9. Signed-off-by: Jamie McCrae <[email protected]>
This reverts commit 9f1e573. Signed-off-by: Jamie McCrae <[email protected]>
Which bug needs to be included out of curiosity? |
Next time, why not create a |
I think that MCUboot requires some definition of release management and proper branching for that. |
The PR was merged by someone that does not follow zephyr releases so was not aware of the zephyr release, it should not have been merged during the zephyr release time. And I'm not fond of creating mirror branches which then deviate from how we use mcuboot in main, for an lts release that's fine or if there is a security fix needed in a still supported version of zephyr but having multiple branches for no reason creates a mess Zephyr x.x.0 releases should always reference a valid mcuboot commit which is in this repository, point releases may diverge |
@nordicjm but MCUboot is its own project, I am not sure why the release cadence and cycles of Zephyr, which is one of its users, should influence the state of |
It is it's own project, the 2 active developers/maintainers are zephyr developers, so the overlap is almost 100% |
These commits should not have been merged until after the zephyr 4.0 stablilisation process was complete, a bug has been found which needs to be included in MCUboot for this release, therefore revert the commits so an MCUboot module update can be included