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

west: runners: add option 'v' prefix to OpenOCD version string #83667

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

EricNRS
Copy link
Contributor

@EricNRS EricNRS commented Jan 7, 2025

The Espressif branch of OpenOCD uses the version string "v0.12.0" instead of "0.12.0" causing the version regex match to fail.

Add an optional 'v' prefix to avoid the failure.

Fixes: #83373

The Espressif branch of OpenOCD uses the version string "v0.12.0" instead
of "0.12.0" causing the version regex match to fail.

Add an optional 'v' prefix to avoid the failure.

Fixes: zephyrproject-rtos#83373
Signed-off-by: Eric Holmberg <[email protected]>
@zephyrbot zephyrbot added size: XS A PR changing only a single line of code area: West West utility labels Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: West West utility size: XS A PR changing only a single line of code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

openocd-esp32 v0.12 does not work with west debug
4 participants