Skip to content

Conversation

@slydiman
Copy link
Contributor

@slydiman slydiman commented Nov 9, 2025

#160199 broke buildbots lldb-remote-linux-ubuntu and lldb-remote-linux-win.
This patch must make these buildbots green for now.

@llvmbot
Copy link
Member

llvmbot commented Nov 9, 2025

@llvm/pr-subscribers-lldb

Author: Dmitry Vasilyev (slydiman)

Changes

#160199 broke buildbots lldb-remote-linux-ubuntu and lldb-remote-linux-win.
This patch must fix these buildbots.


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

1 Files Affected:

  • (modified) lldb/test/API/functionalities/breakpoint/breakpoint_locations/after_rebuild/TestLocationsAfterRebuild.py (+1)
diff --git a/lldb/test/API/functionalities/breakpoint/breakpoint_locations/after_rebuild/TestLocationsAfterRebuild.py b/lldb/test/API/functionalities/breakpoint/breakpoint_locations/after_rebuild/TestLocationsAfterRebuild.py
index 1c7bb538d5df7..f4c91fe9334ca 100644
--- a/lldb/test/API/functionalities/breakpoint/breakpoint_locations/after_rebuild/TestLocationsAfterRebuild.py
+++ b/lldb/test/API/functionalities/breakpoint/breakpoint_locations/after_rebuild/TestLocationsAfterRebuild.py
@@ -19,6 +19,7 @@ class TestLocationsAfterRebuild(TestBase):
 
     # On Windows we cannot remove a file that lldb is debugging.
     @skipIfWindows
+    @skipIfRemote
     def test_remaining_location_spec(self):
         """If we rebuild a couple of times some of the old locations
         get removed.  Make sure the command-line breakpoint id

Copy link
Contributor

@GeorgeHuyubo GeorgeHuyubo left a comment

Choose a reason for hiding this comment

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

Will fix the test on Monday

llvm#160199 broke buildbots `lldb-remote-linux-ubuntu` and `lldb-remote-linux-win`.
This patch must fix these buildbots.
@slydiman slydiman force-pushed the fix-TestLocationsAfterRebuild branch from f011e8e to 7a2f359 Compare November 9, 2025 20:04
@slydiman slydiman merged commit c10b25a into llvm:main Nov 9, 2025
10 checks passed
@slydiman
Copy link
Contributor Author

slydiman commented Nov 9, 2025

Will fix the test on Monday

Thanks. Don't forget to revert this patch with the fix.

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