[Lldb-commits] [PATCH] D13124: test runner: switch to pure-Python timeout mechanism

Todd Fiala via lldb-commits lldb-commits at lists.llvm.org
Mon Sep 28 11:00:40 PDT 2015


tfiala added a comment.

In http://reviews.llvm.org/D13124#254935, @tfiala wrote:

> In http://reviews.llvm.org/D13124#254900, @zturner wrote:
>
> > Sorry, our desks were reconfigured over the weekend, so I just now got my
> >  computer turned on.  I'm syncing code and hopefully will have a working
> >  build soon.
>
>
> Sounds good.  I expect you'll find a few things you'll want to add/adjust.  If you get those patches to me, I'll include them in the check-in.


Also note the test code for this check-in assumes there is a python in the path when it builds the command to run the inferior test subject.  (This is the thing we run against to verify that the ProcessDriver --- aka child process runner with timeout support --- gets return codes, witnesses timeouts, witnesses children processes that choose to ignore soft terminate signals and gets the bad guy with a more lethal (hard) termination option, etc.).  You might need to tweak that in the test runner.  That python invocation doesn't require any special lldb support - it is just testing that a process can be launched, and that process happens to be a python interpreter session running the inferior.py test subject.  So I figured that was probably fine as is.


http://reviews.llvm.org/D13124





More information about the lldb-commits mailing list