Skip to content

Conversation

@jreineckearm
Copy link
Collaborator

@jreineckearm jreineckearm commented Apr 8, 2025

Fixes

Changes

  • Add simple *.cbuild-run.yml reader.
  • Pick SVD file path, replace ${CMSIS_PACK_ROOT} if used in cbuild-run.
  • Set definitionPath (known to Peripheral Inspector).
  • Adds launch configuration to run open Jest test file for debug.

Not possible (see Open-CMSIS-Pack/devtools#2050), address later:

  • Use pname from debugger node to find correct SVD file.
  • Handle multiple debugger nodes.

Screenshots

Checklist

Signed-off-by: Jens Reinecke <[email protected]>
Signed-off-by: Jens Reinecke <[email protected]>
Signed-off-by: Jens Reinecke <[email protected]>
Signed-off-by: Jens Reinecke <[email protected]>
Signed-off-by: Jens Reinecke <[email protected]>
Signed-off-by: Jens Reinecke <[email protected]>
@jreineckearm jreineckearm marked this pull request as ready for review April 11, 2025 13:44
@omarArm
Copy link
Collaborator

omarArm commented Apr 11, 2025

I am trying to test it, but I get this error whenever I try to run a yarn watch

ERROR in ./src/cbuild-run/cbuild-run-reader.ts 52:26-41
Module not found: Error: Can't resolve 'yaml' in '<workspace_folder>/src/cbuild-run'

@jreineckearm
Copy link
Collaborator Author

I am trying to test it, but I get this error whenever I try to run a yarn watch

You may have to run a yarn to pull in a new dependency added with this PR. Best to even yarn clean it first to avoid any conflicting leftovers in node_modules.

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.

Tested it on my machine, everything is working fine

@jreineckearm jreineckearm merged commit 80cb7dd into main Apr 14, 2025
9 checks passed
@jreineckearm jreineckearm deleted the cbuild-run-reader branch April 14, 2025 13:56
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