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

Replace bootutils with mcuboot #1763

Merged
merged 1 commit into from
Apr 17, 2019
Merged

Replace bootutils with mcuboot #1763

merged 1 commit into from
Apr 17, 2019

Conversation

sjanc
Copy link
Contributor

@sjanc sjanc commented Apr 12, 2019

'newt test all' doesn't work due to issues on mcuboot side and need to be fixed before this PR can go in

Initial idea is that core master is using latest mcuboot release (not master), and for core release we stick to latest available version (eg 1.3.0)

There is also a question about newt create-image -2 option... should we change default behavior for newt and always use version 2 now?


pkg.deps.BOOT_SERIAL:
- "@apache-mynewt-core/boot/boot_serial"
pkg.type: transient
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we should remove it at all as it is in the end bit different bootloader.

@utzig
Copy link
Member

utzig commented Apr 12, 2019

There is also a question about newt create-image -2 option... should we change default behavior for newt and always use version 2 now?

After the switch, definitely yes, otherwise create-image would be generating incompatible images! ;-)

@sjanc sjanc force-pushed the mcuboot branch 3 times, most recently from 72a8f26 to 546a7cf Compare April 12, 2019 14:03
@sjanc
Copy link
Contributor Author

sjanc commented Apr 15, 2019

yes, after switch of course :)

Master depends on latest released mcuboot (1-latest), when doing
releases release branch will depend on specific version eg 1.3
which is latest available at the release time.
Copy link
Contributor

@rymanluk rymanluk left a comment

Choose a reason for hiding this comment

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

lgtm

@sjanc sjanc merged commit 540ee18 into apache:master Apr 17, 2019
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.

None yet

3 participants