[Lldb-commits] [PATCH] D67173: [dotest] Centralize initialization commands even more

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Sep 4 07:30:58 PDT 2019


labath created this revision.
labath added reviewers: JDevlieghere, jankratochvil.

In r367234 we introduced a central place to hold the set up commands for
the various ways we have of launching lldb. However, a number of
commands still remained outside of that.

This patch moves the remaining set up commands into this function, which
allows us to remove manual clang module path setting code in
TestBatchMode.

One unfortunate victim of this approach is TestSTTYBeforeAndAfter which,
due to how it launches lldb (pexpect->expect->lldb), fails get the
quoting right. It would be possible to fix the quoting there, it would be a bit
icky, and none of the commands in this list are really relevant for what this
test is doing, so I just remove the commands outright.


https://reviews.llvm.org/D67173

Files:
  packages/Python/lldbsuite/test/driver/batch_mode/TestBatchMode.py
  packages/Python/lldbsuite/test/lldbtest.py
  packages/Python/lldbsuite/test/terminal/TestSTTYBeforeAndAfter.py
  packages/Python/lldbsuite/test/tools/lldb-vscode/lldbvscode_testcase.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67173.218694.patch
Type: text/x-patch
Size: 5886 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20190904/1456f71b/attachment.bin>


More information about the lldb-commits mailing list