[Lldb-commits] [PATCH] D25486: Fix lookup path for lldb-mi

Chris Bieneman via lldb-commits lldb-commits at lists.llvm.org
Wed Oct 12 13:53:55 PDT 2016

beanz added inline comments.

Comment at: packages/Python/lldbsuite/test/dotest.py:676-677
     # If not found, disable the lldb-mi tests
-    lldbMiExec = None
-    if lldbtest_config.lldbExec and is_exe(lldbtest_config.lldbExec + "-mi"):
-        lldbMiExec = lldbtest_config.lldbExec + "-mi"
-    if not lldbMiExec:
+    lldbDir = os.path.dirname(lldbtest_config.lldbExec)
+    lldbMiExec = os.path.join(lldbDir, "lldb-mi")
+    if is_exe(lldbMiExec):
ki.stfu wrote:
> Maybe it would be better to replace "lldb" with "lldb-mi" in lldbtest_config.lldbExec? So we will take into account their versions instead of ignoring them.
That is a good idea, assuming the patch as-is doesn't break the world, I'll revise it to that to fix the Windows issue as well.



More information about the lldb-commits mailing list