Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[lldb][test] Temporarily disable TestQueueFromStdModule.py #68970

Merged

Conversation

Michael137
Copy link
Member

Started failing since D101206, but root-cause is unclear. It's definitely not an issue with th libc++ patch itself however. So disable the test until we know what's going on.

@llvmbot
Copy link
Member

llvmbot commented Oct 13, 2023

@llvm/pr-subscribers-lldb

Author: Michael Buch (Michael137)

Changes

Started failing since D101206, but root-cause is unclear. It's definitely not an issue with th libc++ patch itself however. So disable the test until we know what's going on.


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

1 Files Affected:

  • (modified) lldb/test/API/commands/expression/import-std-module/queue/TestQueueFromStdModule.py (+1)
diff --git a/lldb/test/API/commands/expression/import-std-module/queue/TestQueueFromStdModule.py b/lldb/test/API/commands/expression/import-std-module/queue/TestQueueFromStdModule.py
index 84e8e3cfb86d6fd..5f344a4f36224db 100644
--- a/lldb/test/API/commands/expression/import-std-module/queue/TestQueueFromStdModule.py
+++ b/lldb/test/API/commands/expression/import-std-module/queue/TestQueueFromStdModule.py
@@ -10,6 +10,7 @@
 class TestQueue(TestBase):
     @add_test_categories(["libc++"])
     @skipIf(compiler=no_match("clang"))
+    @skipIf(compiler="clang", compiler_version=[">", "16.0"], bugnumber="https://github.com/llvm/llvm-project/issues/68968")
     def test(self):
         self.build()
 

@github-actions
Copy link

github-actions bot commented Oct 13, 2023

✅ With the latest revision this PR passed the Python code formatter.

Started failing since D101206 (`ba79fb2e1ff7130cde02fbbd325f0f96f8a522ca`),
but root-cause is unclear. It's definitely not an issue with th libc++ patch
itself however. So disable the test until we know what's going on.
@Michael137 Michael137 force-pushed the bugfix/lldb-import-queue-test-disable branch from ae9c1d1 to d358829 Compare October 13, 2023 15:10
@Michael137 Michael137 merged commit 74c5e47 into llvm:main Oct 13, 2023
@Michael137 Michael137 deleted the bugfix/lldb-import-queue-test-disable branch October 13, 2023 15:26
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.

3 participants