JDevlieghere marked 2 inline comments as done.
JDevlieghere added inline comments.


================
Comment at: 
lldb/packages/Python/lldbsuite/test/functionalities/thread/concurrent_events/TestConcurrentManyBreakpoints.py:18
     @expectedFailureNetBSD
-    @skipIfOutOfTreeDebugserver
+    @skipIfOutOfTreeDebugserver(['<=', '9999'])
     def test(self):
----------------
labath wrote:
> JDevlieghere wrote:
> > I used 9999 here because that's commonly the placeholder until the next 
> > release. 
> I haven't looked at the code in detail, but given that you are removing the 
> explicit notion of an "out of tree" debug server, it seems to be that this 
> will kick in even if you are testing against the in-tree debug server, which 
> doesn't seem right. Or am I missing something?
Yeah, I was just going to check for version numbers below a hundred, but then 
forgot to add it before I uploaded the patch. 


Repository:
  rLLDB LLDB

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68003/new/

https://reviews.llvm.org/D68003



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to