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

sysbuild: Fix board revisions not being passed to target images #52369

Merged
merged 1 commit into from
Jan 12, 2023

Conversation

nordicjm
Copy link
Collaborator

Fixes an issue whereby board revisions were not being passed to target images and adds a new flag that allows passing the board revision to target images for different target boards but preserving the board revision.

Fixes #52353

Copy link
Collaborator

@tejlmand tejlmand left a comment

Choose a reason for hiding this comment

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

something related to sysbuild cache sharing looks wrong.
Taking a look.

share/sysbuild/cmake/modules/sysbuild_extensions.cmake Outdated Show resolved Hide resolved
@tejlmand
Copy link
Collaborator

tejlmand commented Dec 8, 2022

@nordicjm fixed the root cause here: #52884

This PR is still valid, as it provides a mechanism for project to specify not only board but also revision, however as mentioned #52369 (comment) the else part should not be needed.

@nordicjm nordicjm requested review from tejlmand and Cherish-Gww and removed request for Cherish-Gww December 12, 2022 12:16
Copy link
Collaborator

@tejlmand tejlmand left a comment

Choose a reason for hiding this comment

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

small nits noticed.

@nordicjm nordicjm requested review from tejlmand and removed request for Cherish-Gww December 20, 2022 13:59
Allows specifying board revisions when configuring target images
with sysbuild.

Signed-off-by: Jamie McCrae <[email protected]>
@carlescufi carlescufi merged commit ce31799 into zephyrproject-rtos:main Jan 12, 2023
@nordicjm nordicjm deleted the sysbuild_revision_fix branch August 14, 2023 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

bug: sysbuild lost board reversion here
4 participants