Skip to content

Commit 82ab359

Browse files
authored
[lldb-dap] Fix machine-overridable scope (#137723)
My previous commit was a bit sloppy, I didn't spot that `lldb-dap.executable-path` already had a scope and I didn't consider that `lldb-dap.log-path` should have the same scope. This PR addresses both: - Fix duplicate scope for `lldb-dap.executable-path`. - Change scope of `lldb-dap.log-path` to `machine-overridable` too.
1 parent e4d2ff5 commit 82ab359

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

lldb/tools/lldb-dap/package.json

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -73,9 +73,8 @@
7373
"title": "lldb-dap",
7474
"properties": {
7575
"lldb-dap.executable-path": {
76-
"scope": "resource",
77-
"type": "string",
7876
"scope": "machine-overridable",
77+
"type": "string",
7978
"description": "The path to the lldb-dap binary, e.g. /usr/local/bin/lldb-dap"
8079
},
8180
"lldb-dap.arguments": {
@@ -88,7 +87,7 @@
8887
"description": "The list of additional arguments used to launch the debug adapter executable."
8988
},
9089
"lldb-dap.log-path": {
91-
"scope": "resource",
90+
"scope": "machine-overridable",
9291
"type": "string",
9392
"description": "The log path for lldb-dap (if any)"
9493
},

0 commit comments

Comments
 (0)