Skip to content

scripts: runners: openocd: Fix empty search path #38300

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

Conversation

nandojve
Copy link
Member

@nandojve nandojve commented Sep 3, 2021

When enable thread awareness feature for OpenOCD the search path was converted to a list. In some environments OPENOCD_DEFAULT_PATH may not be defined. That create an empty search path list system fails. This add a test to skips fill search_args with openocd_search values when list is empty.

Fixes #38272.

Signed-off-by: Gerson Fernando Budke [email protected]

When enable thread awareness feature for OpenOCD the search path was
converted to a list.  In some environments OPENOCD_DEFAULT_PATH may
not be defined.  That create an empty search path list system fails.
This add a test to skips fill search_args with openocd_search values
when list is empty.

Fixes zephyrproject-rtos#38272.

Signed-off-by: Gerson Fernando Budke <[email protected]>
@nandojve nandojve requested review from erwango and gmarull September 3, 2021 15:17
@nandojve nandojve added this to the v2.7.0 milestone Sep 3, 2021
@nandojve nandojve added bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug area: West West utility area: OpenOCD labels Sep 3, 2021
@nwsetec
Copy link
Contributor

nwsetec commented Sep 7, 2021

Trying this PR out fixes the #38272 issue for me.

@nashif nashif merged commit 7bf00c9 into zephyrproject-rtos:main Sep 9, 2021
@nandojve nandojve deleted the zephyr_openocd_fix_empty_search_path branch September 10, 2021 02:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: OpenOCD area: West West utility bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"west flash" stopped working
7 participants