From 5c85ea5355577307aab65823589fc60f27855794 Mon Sep 17 00:00:00 2001 From: Jonas Devlieghere Date: Mon, 28 Apr 2025 15:21:16 -0700 Subject: [PATCH] [lldb-dap] Fix machine-overridable scope 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. --- lldb/tools/lldb-dap/package.json | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) 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)" },