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.

Sign up for GitHub

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

Jump to bottom

Update the instruction for debugging #1

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Update the instruction for debugging #1

wants to merge 2 commits into from

Conversation

piyush-saurabh
Copy link

Couple of instructions needs to be updated.

  1. "Native Debug" vscode extension is required for "type": "gdb" in launch.json. Otherwise IDE will show squiggly lines and debugging will not start
  2. On Ubuntu 20.4, pressing "ctrl+alt+t" inside vscode (default installation) will open terminal instead of task list. Updated the instruction accordingly.

@piyush-saurabh
Update README.md
4c30eac 
added instruction to install vscode gdb extension
@piyush-saurabh
Update README.md
9c582db 
updated instruction to execute tasks on default vscode installation
sidwarkd
sidwarkd requested changes Apr 17, 2021
View reviewed changes
Copy link
Contributor

@sidwarkd sidwarkd left a comment

Choose a reason for hiding this comment

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

Thanks so much for making the docs better. Just a minor wording change to be explicit on what Ctrl-Shift-P opens and then I'll merge.

README.md
1. Install the VS Code GDB extension https://marketplace.visualstudio.com/items?itemName=webfreak.debug
2. Attach the ESP-Prog to your target device via whatever JTAG cable is required.
3. Plug the target device (the one you want to debug) into your computer.
4. Press **Ctrl-Shift-P** to open the Task list in VS Code. Type 'Run Task' and press enter. Now select _Flash and Monitor Device_.
Copy link
Contributor

Choose a reason for hiding this comment

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

Ctrl-Shift-P opens the Command Palette instead of the Task List. Let's change the wording to say that instead of "Task List" now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sidwarkd sidwarkd sidwarkd requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@piyush-saurabh @sidwarkd