Skip to content

Conversation

@slydiman
Copy link
Contributor

Skip the test for Windows hosts.
This patch fixes the buildbot lldb-remote-linux-win.
https://lab.llvm.org/buildbot/#/builders/197/builds/10304

Skip the test for Windows hosts.
This patch fixes the buildbot `lldb-remote-linux-win`.
https://lab.llvm.org/buildbot/#/builders/197/builds/10304
@llvmbot
Copy link
Member

llvmbot commented Oct 28, 2025

@llvm/pr-subscribers-lldb

Author: Dmitry Vasilyev (slydiman)

Changes

Skip the test for Windows hosts.
This patch fixes the buildbot lldb-remote-linux-win.
https://lab.llvm.org/buildbot/#/builders/197/builds/10304


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

1 Files Affected:

  • (modified) lldb/test/API/driver/stdio_closed/TestDriverWithClosedSTDIO.py (+1-1)
diff --git a/lldb/test/API/driver/stdio_closed/TestDriverWithClosedSTDIO.py b/lldb/test/API/driver/stdio_closed/TestDriverWithClosedSTDIO.py
index 13437d05557bf..a73322c78d81e 100644
--- a/lldb/test/API/driver/stdio_closed/TestDriverWithClosedSTDIO.py
+++ b/lldb/test/API/driver/stdio_closed/TestDriverWithClosedSTDIO.py
@@ -24,7 +24,7 @@ class TestDriverWithClosedSTDIO(TestBase):
 
     # Windows doesn't have the fcntl module, so we can't run this
     # test there.
-    @skipIf(oslist=["windows"])
+    @skipIf(hostoslist=["windows"])
     def test_run_lldb_and_wait(self):
         """This test forks, closes the stdio channels and exec's lldb.
         Then it waits for it to exit and asserts it did that successfully"""

Copy link
Collaborator

@DavidSpickett DavidSpickett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Before: skip if lldb tests are run on a Windows machine
After: skip if the test itself runs on a Windows machine (build machine or remote)

Sounds good to me.

@slydiman slydiman enabled auto-merge (squash) October 28, 2025 15:35
@slydiman slydiman merged commit 624d4f6 into llvm:main Oct 28, 2025
13 checks passed
aokblast pushed a commit to aokblast/llvm-project that referenced this pull request Oct 30, 2025
DEBADRIBASAK pushed a commit to DEBADRIBASAK/llvm-project that referenced this pull request Nov 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants