Author: Jonas Devlieghere Date: 2025-04-28T17:01:03-07:00 New Revision: 82ab35961f62c8019f3b0129da42c42287a557b0
URL: https://github.com/llvm/llvm-project/commit/82ab35961f62c8019f3b0129da42c42287a557b0 DIFF: https://github.com/llvm/llvm-project/commit/82ab35961f62c8019f3b0129da42c42287a557b0.diff LOG: [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. Added: Modified: lldb/tools/lldb-dap/package.json Removed: ################################################################################ diff --git a/lldb/tools/lldb-dap/package.json b/lldb/tools/lldb-dap/package.json index adbb5b2bcf722..5cfd79849b17f 100644 --- a/lldb/tools/lldb-dap/package.json +++ b/lldb/tools/lldb-dap/package.json @@ -73,9 +73,8 @@ "title": "lldb-dap", "properties": { "lldb-dap.executable-path": { - "scope": "resource", - "type": "string", "scope": "machine-overridable", + "type": "string", "description": "The path to the lldb-dap binary, e.g. /usr/local/bin/lldb-dap" }, "lldb-dap.arguments": { @@ -88,7 +87,7 @@ "description": "The list of additional arguments used to launch the debug adapter executable." }, "lldb-dap.log-path": { - "scope": "resource", + "scope": "machine-overridable", "type": "string", "description": "The log path for lldb-dap (if any)" }, _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
