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

[Ilia K]

Vince,

How you think?

Thanks,
Ilia

On Fri, Feb 27, 2015 at 8:06 AM, Ilia K <ki.stfu at gmail.com> wrote:

> 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/20150304/4a4b22d6/attachment.html>