[Lldb-commits] [lldb] r230466 - Fix ImportError in lldb-mi tests on Linux

[Ilia K]

Hello Vince,

Sorry for delay. We can add lldb-mi folder to PATH in dotest.py (by
default) and then we be able to roll back these changes. It will look like:

> Index: test/dotest.py
>
===================================================================

--- test/dotest.py      (revision 230655)

+++ test/dotest.py      (working copy)

@@ -932,14 +932,16 @@


>      pluginPath = os.path.join(scriptPath, 'plugins')

     pexpectPath = os.path.join(scriptPath, 'pexpect-2.4')

+    toolsLldbMiPath = os.path.join(scriptPath, 'tools', 'lldb-mi')


>      # Put embedded pexpect at front of the load path so we ensure we

     # use that version.

     sys.path.insert(0, pexpectPath)


> -    # Append script dir and plugin dir to the sys.path.

+    # Append script dir, plugin dir and lldb-mi dir to the sys.path.

     sys.path.append(scriptPath)

     sys.path.append(pluginPath)

+    sys.path.append(toolsLldbMiPath)


>      # This is our base name component.

     base = os.path.abspath(os.path.join(scriptPath, os.pardir))


Does it look better?

Thanks,
Ilia


On Wed, Feb 25, 2015 at 3:18 PM, Vince Harron <vharron at google.com> wrote:

> Is it possible to have the application search for the needed binaries in
> specific locations instead of requiring the user to add them to the path?
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150227/da3a1611/attachment.html>