Skip to content

west: fix for context with sysbuild #87910

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 25, 2025
Merged

west: fix for context with sysbuild #87910

merged 1 commit into from
Apr 25, 2025

Conversation

vignesh-aerlync
Copy link
Contributor

show west flash --context with sysbuild

@github-actions github-actions bot added the area: West West utility label Mar 31, 2025
@vignesh-aerlync vignesh-aerlync mentioned this pull request Mar 31, 2025
Closed
@pdgendt pdgendt requested a review from nordicjm March 31, 2025 14:13
@carlescufi carlescufi requested a review from tejlmand March 31, 2025 14:30
carlescufi
carlescufi requested changes Mar 31, 2025
View reviewed changes
@vignesh-aerlync vignesh-aerlync force-pushed the west_context_sysbuild branch 3 times, most recently from 39b2236 to 3fe5203 Compare April 2, 2025 02:05
@vignesh-aerlync
Copy link
Contributor Author

#87497
To link issue with PR.

@pdgendt pdgendt linked an issue Apr 7, 2025 that may be closed by this pull request
west flash --context fails for Sysbuild projects #87497
Closed
@vignesh-aerlync
Copy link
Contributor Author

Hi @nordicjm @carlescufi gentle reminder to review this PR when you get a chance. Let me know if any changes are needed. Thanks!

nordicjm
nordicjm previously approved these changes Apr 14, 2025
View reviewed changes
Copy link
Collaborator

@nordicjm nordicjm left a comment

Choose a reason for hiding this comment

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

Works well!

@pdgendt
Copy link
Collaborator

pdgendt commented Apr 14, 2025

@carlescufi PTAL

@vignesh-aerlync
west: fix for context with sysbuild
6294ba3 
show `west flash --context` with sysbuild

Signed-off-by: Vignesh Pandian <[email protected]>
@pdgendt pdgendt assigned carlescufi and unassigned pdgendt Apr 23, 2025
@kartben kartben merged commit 8972146 into zephyrproject-rtos:main Apr 25, 2025
41 checks passed
@github-actions GitHub Actions
Copy link

Hi @vignesh-aerlync!
Congratulations on getting your very first Zephyr pull request merged 🎉🥳. This is a fantastic achievement, and we're thrilled to have you as part of our community!

To celebrate this milestone and showcase your contribution, we'd love to award you the Zephyr Technical Contributor badge. If you're interested, please claim your badge by filling out this form: Claim Your Zephyr Badge.

Thank you for your valuable input, and we look forward to seeing more of your contributions in the future! 🪁

@brycewilkins
Copy link
Contributor

Thank you @vignesh-aerlync for resolution to the problem! Great work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tejlmand tejlmand tejlmand approved these changes

@carlescufi carlescufi carlescufi approved these changes

@nordicjm nordicjm nordicjm approved these changes

@swinslow swinslow Awaiting requested review from swinslow

@pdgendt pdgendt Awaiting requested review from pdgendt

Assignees

@carlescufi carlescufi

Labels
area: West West utility
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

west flash --context fails for Sysbuild projects
7 participants
@vignesh-aerlync @pdgendt @brycewilkins @tejlmand @carlescufi @nordicjm @kartben