Skip to content

[lldb-dap][windows] fix invalid path substitution for lldb-dap#180237

Merged
charles-zablit merged 1 commit into
llvm:mainfrom
charles-zablit:charles-zablit/lldb/fix-dap-test
Feb 9, 2026
Merged

[lldb-dap][windows] fix invalid path substitution for lldb-dap#180237
charles-zablit merged 1 commit into
llvm:mainfrom
charles-zablit:charles-zablit/lldb/fix-dap-test

Conversation

@charles-zablit
Copy link
Copy Markdown
Contributor

#178642 added lldb/test/Shell/DAP/TestSTDINConsole.test with an incorrect %lldb-dap expansion. This patch fixes it.

@charles-zablit charles-zablit self-assigned this Feb 6, 2026
@llvmbot llvmbot added the lldb label Feb 6, 2026
@llvmbot
Copy link
Copy Markdown
Member

llvmbot commented Feb 6, 2026

@llvm/pr-subscribers-lldb

Author: Charles Zablit (charles-zablit)

Changes

#178642 added lldb/test/Shell/DAP/TestSTDINConsole.test with an incorrect %lldb-dap expansion. This patch fixes it.


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

1 Files Affected:

  • (modified) lldb/test/Shell/DAP/TestSTDINConsole.test (+1-1)
diff --git a/lldb/test/Shell/DAP/TestSTDINConsole.test b/lldb/test/Shell/DAP/TestSTDINConsole.test
index ba2357d1b0fe4..0d6d25400260e 100644
--- a/lldb/test/Shell/DAP/TestSTDINConsole.test
+++ b/lldb/test/Shell/DAP/TestSTDINConsole.test
@@ -1,5 +1,5 @@
 # REQUIRES: python, system-windows
-# RUN: %python %s %lldb-dap > %t.out 2>&1
+# RUN: %python %s lldb-dap > %t.out 2>&1
 # RUN: FileCheck %s --check-prefix=ERROR --allow-empty < %t.out
 
 # ERROR-NOT: DAP session error:

@charles-zablit charles-zablit merged commit dbbea96 into llvm:main Feb 9, 2026
12 checks passed
charles-zablit added a commit to charles-zablit/llvm-project that referenced this pull request Feb 9, 2026
…180237)

llvm#178642 added
`lldb/test/Shell/DAP/TestSTDINConsole.test` with an incorrect
`%lldb-dap` expansion. This patch fixes it.

(cherry picked from commit dbbea96)
rishabhmadan19 pushed a commit to rishabhmadan19/llvm-project that referenced this pull request Feb 9, 2026
…180237)

llvm#178642 added
`lldb/test/Shell/DAP/TestSTDINConsole.test` with an incorrect
`%lldb-dap` expansion. This patch fixes it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants