================
@@ -60,6 +61,15 @@ export class LLDBDapServer implements vscode.Disposable {
     }
 
     this.serverInfo = new Promise((resolve, reject) => {
+      if (os.platform() === "win32") {
+        const pythonCheckProcess = child_process.spawnSync(dapPath, 
["--check-python"]);
----------------
charles-zablit wrote:

Maybe I should have marked this PR as a draft.

There are a couple of questions which are still unanswered by this patch:
1. What happens if the `lldb-dap` binary is too old and `--check-python` does 
not exist yet? Should we first run `lldb-dap.exe --version`, parse it and make 
sure it's recent enough? But that does not work for `main` builds.
2. As you said, what happens if `lldb-dap` is not built with Python support? We 
could parse a predefined error like "was not built with Python support".

https://github.com/llvm/llvm-project/pull/181124
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to