llvmbot wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: Ebuka Ezike (da-viper)

<details>
<summary>Changes</summary>

There is no indication this ever worked on windows. 

Looking at the error from the CI it closed the interpreter before running any 
commands. 
Will reconfirm this tomorrow when I have access to a windows machine. 

>From https://github.com/llvm/llvm-project/pull/174216

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


2 Files Affected:

- (modified) lldb/test/API/python_api/file_handle/TestFileHandle.py (+1) 
- (modified) lldb/test/Shell/ScriptInterpreter/Python/io.test (+2) 


``````````diff
diff --git a/lldb/test/API/python_api/file_handle/TestFileHandle.py 
b/lldb/test/API/python_api/file_handle/TestFileHandle.py
index 707044a3afb0f..b69724ec8db11 100644
--- a/lldb/test/API/python_api/file_handle/TestFileHandle.py
+++ b/lldb/test/API/python_api/file_handle/TestFileHandle.py
@@ -680,6 +680,7 @@ def test_stdout_file(self):
             lines = [x for x in f.read().strip().split() if x != "7"]
             self.assertEqual(lines, ["foobar"])
 
+    @skipIfWindows
     def test_stdout_file_interactive(self):
         """Ensure when we read stdin from a file, outputs from python goes to 
the right I/O stream."""
         with open(self.in_filename, "w") as f:
diff --git a/lldb/test/Shell/ScriptInterpreter/Python/io.test 
b/lldb/test/Shell/ScriptInterpreter/Python/io.test
index 25e3de41724e0..1a3ff8dcd4258 100644
--- a/lldb/test/Shell/ScriptInterpreter/Python/io.test
+++ b/lldb/test/Shell/ScriptInterpreter/Python/io.test
@@ -1,3 +1,5 @@
+# UNSUPPORTED: system-windows
+
 # RUN: rm -rf %t.stdout %t.stderr
 # RUN: cat %s | %lldb --script-language python > %t.stdout 2> %t.stderr
 # RUN: cat %t.stdout | FileCheck %s --check-prefix STDOUT

``````````

</details>


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

Reply via email to