Skip to content

Conversation

@jreineckearm
Copy link
Collaborator

@jreineckearm jreineckearm commented Feb 20, 2025

Fixes

N/A

Changes

  • Removed pyOCD auto-fill of --target and --pack. This will be obsolete with the first pyOCD dev drops that use --cbuild-run.
  • Resolve with substituted variables for pyOCD now that the commands for the two above pyOCD CLI args are no longer needed. Moves resolving to the latest possible point in time.
  • Use future pyOCD --cbuild-run CLI arg if debug launch config item provided. NOTE: Remove cbuildRunFile setting if using an older pyOCD version. Otherwise launching it will fail.
  • Prepend auto-filled gdbserver arg for pyOCD. Must be first argument. Relevant if mixing user provided server args with auto-filled ones.
  • Removed reset before load in pyOCD init commands.
  • More refactoring.
  • README update to explain current setup steps for pyOCD debug (for when pyOCD is included in extension).

Screenshots

Checklist

Copy link
Collaborator

@omarArm omarArm left a comment

Choose a reason for hiding this comment

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

I think that the best course is to halt the device using monitor halt and then load. Obviously if there was no pre-existing image on the device already, then no need to halt

@jreineckearm jreineckearm merged commit 80a006d into main Feb 21, 2025
1 check passed
@jreineckearm jreineckearm deleted the fix-pyocd-launch branch February 21, 2025 10:49
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.

3 participants