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

Add --gdb argument for debugging #47

Merged
merged 2 commits into from
Jan 17, 2025

Conversation

piotrnarajowski
Copy link
Contributor

@piotrnarajowski piotrnarajowski commented Nov 29, 2024

This commit implements --gdb argument. Reset is skipped for central/peripheral/both board(s) when btptester.py is launched. This might be helpful for debugging. Also, in btptester.py added rerun_failed variable initialization. This fixes access to uninitialized variable

@piotrnarajowski piotrnarajowski force-pushed the add_gdb_arg branch 2 times, most recently from 2f01862 to 6ca9245 Compare December 12, 2024 14:47
btptester.py Outdated
help='Stops the run at first failure'),
parser.add_argument('--gdb', type=str,
help="Skip selected board reset to avoid gdb server"
" disconnection e.g. --gdb central/prph/both")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nitpick: I'd say, either use short or full names, so either central/peripheral or cent/perph

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Switched to short

This commit implements --gdb argument. Board reset will be skipped for
central/peripheral/both when btptester.py is launched.
This might be helpful for debugging.
This fixes access to uninitialized variable
Copy link
Collaborator

@KKopyscinski KKopyscinski left a comment

Choose a reason for hiding this comment

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

LGTM

@KKopyscinski KKopyscinski merged commit 2f6db31 into JuulLabs-OSS:master Jan 17, 2025
@piotrnarajowski piotrnarajowski deleted the add_gdb_arg branch January 17, 2025 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants