[Lldb-commits] [PATCH] D68727: Allow pexpect tests to work in remote testsuite runs

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Oct 9 23:52:57 PDT 2019


labath accepted this revision.
labath added a comment.
This revision is now accepted and ready to land.

I don't see harm in adding this, but I am curious about what are you planning to use this for. My takeaway from all of the pexpect discussions we've had in the past was that pexpect should only be used for stuff that really, really needs an interactive terminal to work (i.e., stuff like batch mode, the "gui" command, interactive command line editing/libedit stuff, etc.). All of these seem like innately host-related stuff, and I am having trouble imagining a feature/test that requires an interactive terminal, but only works on remote targets.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68727/new/

https://reviews.llvm.org/D68727





More information about the lldb-commits mailing list