[dev] vscode debug launch.json #1537
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1537 +/- ##
==========================================
- Coverage 84.63% 84.57% -0.06%
==========================================
Files 156 156
Lines 4795 4795
==========================================
- Hits 4058 4055 -3
- Misses 737 740 +3
|
|
Should we commit IDE setting to repository?Different users use different IDE and toolchains. |
We discussed this PR in the SIG meeting. I think it is good to at least have something minimal if everybody agrees of course. |
LGTM. I don't see how this can hurt, and it will increase our productivity. |
Looks like OpenTelemetry C++ is not the first one https://github.com/open-telemetry/opentelemetry-specification/blob/main/.vscode/settings.json. |
.vscode/launch.json
Outdated
| "valuesFormatting": "parseText" | ||
| } | ||
| ] | ||
| } |
There was a problem hiding this comment.
nit: consider adding a blank new line here
| "name": "Debug", | ||
| "type": "gdb", | ||
| "request": "launch", | ||
| "target": "${workspaceFolder}/bazel-bin/<path to the bin file>", |
There was a problem hiding this comment.
This is bazel specific and some some manual update for CMake?
There was a problem hiding this comment.
it can be used with CMake too, just need to change the target path.
| "cwd": "${workspaceRoot}", | ||
| "valuesFormatting": "parseText" | ||
| } | ||
| ] |
There was a problem hiding this comment.
Should we add the configuration for Windows too? Something like this should work.
{
"name": "Debug on Windows",
"type": "cppvsdbg",
"request": "launch",
"program": "${workspaceFolder}/build/<path-to-bin-file>",
"args": [],
"stopAtEntry": false,
"cwd": "${workspaceFolder}",
"environment": [],
"externalConsole": false
}
Co-authored-by: Lalit Kumar Bhasin <lalit_fin@yahoo.com>
Adds vscode launch.json
launch.json is needed for debugging in VS code.
Changes
how to use:
build in debug mode e.g. bazel:
--compilation_mode=dbgor CMake with:
-DCMAKE_BUILD_TYPE=Debugchange the target to the executable of interest e.g.
curl_http_test:"target": "${workspaceFolder}/bazel-bin/ext/test/http/curl_http_test"For significant contributions please make sure you have completed the following items:
CHANGELOG.mdupdated for non-trivial changes