[Lldb-commits] [PATCH] D29985: [lldb] [test] Fix finding LLDB tools when building stand-alone

[Pavel Labath via Phabricator via lldb-commits]

labath added a comment.

It does not seem too controversial, but I am not very confident reviewing lit changes. @beanz, could you take a look at this?



================
Comment at: lit/lit.cfg:124
 
-debugserver = lit.util.which('debugserver', llvm_tools_dir)
-lldb = lit.util.which('lldb', llvm_tools_dir)
+debugserver = lit.util.which('debugserver', config.environment['PATH'])
+lldb = lit.util.which('lldb', config.environment['PATH'])
----------------
Wouldn't it be better to use lldb_tools_dir instead of the PATH here? Less chance to pick up the wrong tool accidentally?


Repository:
  rL LLVM

https://reviews.llvm.org/D29985