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

runners: jlink: Fix NoneType object error #79889

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

ngphibang
Copy link
Contributor

The commit 221199e presents a bug that makes west flash failed with error.

AttributeError: 'NoneType' object has no attribute 'startswith'

In function is_tunnel(), tunnel may contain None and has no attribute "startswith".

The commit 221199e presents a bug that
makes west flash failed with error.

AttributeError: 'NoneType' object has no attribute 'startswith'

In function is_tunnel(), tunnel may contain None and has no attribute
"startswith". Fix it.

Signed-off-by: Phi Bang Nguyen <[email protected]>
@zephyrbot zephyrbot added size: XS A PR changing only a single line of code area: West West utility labels Oct 15, 2024
@dleach02 dleach02 added the Regression Something, which was working, does not anymore label Oct 15, 2024
@dleach02
Copy link
Member

attn: @adamdunkels

@adamdunkels
Copy link

attn: @adamdunkels

looks good!

@ngphibang
Copy link
Contributor Author

ngphibang commented Oct 15, 2024

CI error seems unrelated to the change:

INFO - 590/736 nrf54h20dk/nrf54h20/cpuapp tests/drivers/pwm/pwm_api/drivers.pwm.pwm_fast ERROR Build failure (build)

Copy link
Collaborator

@kartben kartben left a comment

Choose a reason for hiding this comment

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

Nite: it feels like it would be clearer if the parameter to is_ip and is_tunnel would be named dev_id. Seems odd to check if a tunnel is a tunnel or if an IP address is an IP address :)

@dleach02 dleach02 added the Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. label Oct 16, 2024
@butok
Copy link
Contributor

butok commented Oct 16, 2024

CI error seems unrelated to the change:

INFO - 590/736 nrf54h20dk/nrf54h20/cpuapp tests/drivers/pwm/pwm_api/drivers.pwm.pwm_fast ERROR Build failure (build)

Global CI issue, new PRs has this CI error now.

@pdgendt
Copy link
Collaborator

pdgendt commented Oct 16, 2024

CI error seems unrelated to the change:
INFO - 590/736 nrf54h20dk/nrf54h20/cpuapp tests/drivers/pwm/pwm_api/drivers.pwm.pwm_fast ERROR Build failure (build)

Global CI issue, new PRs has this CI error now.

Has been fixed in #79904, rebase on main

@dleach02 dleach02 added the Hotfix Fix for issues blocking development, i.e. CI issues, tests failing in CI, etc. label Oct 16, 2024
@dleach02 dleach02 merged commit 829c03b into zephyrproject-rtos:main Oct 16, 2024
46 checks passed
@dleach02 dleach02 deleted the fix_jlink_error branch October 16, 2024 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: West West utility Hotfix Fix for issues blocking development, i.e. CI issues, tests failing in CI, etc. Regression Something, which was working, does not anymore size: XS A PR changing only a single line of code Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants