llvmbot wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: Charles Zablit (charles-zablit)

<details>
<summary>Changes</summary>

This patch fixes the `lldb-dap` executable substitution in tests. This was not 
done before.

---
Full diff: https://github.com/llvm/llvm-project/pull/179685.diff


1 Files Affected:

- (modified) lldb/test/Shell/helper/toolchain.py (+3-1) 


``````````diff
diff --git a/lldb/test/Shell/helper/toolchain.py 
b/lldb/test/Shell/helper/toolchain.py
index 66664561a249d..5803ecb8d96c1 100644
--- a/lldb/test/Shell/helper/toolchain.py
+++ b/lldb/test/Shell/helper/toolchain.py
@@ -167,7 +167,9 @@ def use_lldb_substitutions(config):
             unresolved="ignore",
         ),
         "lldb-test",
-        "lldb-dap",
+        ToolSubst(
+            "%lldb-dap", command=FindTool("lldb-dap"), extra_args=[], 
unresolved="fatal"
+        ),
         ToolSubst(
             "%build", command="'" + sys.executable + "'", 
extra_args=build_script_args
         ),

``````````

</details>


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

Reply via email to