[Lldb-commits] [PATCH] [TestLoadUnload] Enable for Android while skipping it for other remotes.

Tamas Berghammer tberghammer at google.com
Tue Jun 16 18:04:16 PDT 2015

Why do you enable these tests only on remote Android? I think they should work on any remote target with your fix (where the test makes sense)

Comment at: test/functionalities/load_unload/TestLoadUnload.py:59
@@ +58,3 @@
+                err = lldb.remote_platform.Run(
+                    lldb.SBPlatformShellCommand("mkdir %s" % hidden_dir))
+                if err.Fail():
I think this will fail in case of Windows -> Android (wrong path separator). You can use SBPlatform::MakeDirectory what should work in all cases

Comment at: test/functionalities/load_unload/TestLoadUnload.py:177
@@ +176,3 @@
+            err = lldb.remote_platform.Run(
+                lldb.SBPlatformShellCommand("rm %s" % old_dylib))
+            if err.Fail():
Same here



More information about the lldb-commits mailing list