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

Todd Fiala via lldb-commits lldb-commits at lists.llvm.org
Wed Sep 23 23:08:40 PDT 2015


tfiala created this revision.
tfiala added reviewers: zturner, clayborg, labath.
tfiala added a subscriber: lldb-commits.

For all the parallel test runners, provide a pure-Python mechanism for timing out dotest inferior processes that run for too long.

Stock OS X and Windows do not have a built-in timeout helper app.  This allows those systems to support timeouts.  It also transforms the timeout from being implemented via a system process to costing a single thread.

Tested on Linux and OS X with positive timeout verification.

Greg: this also comments out one of the curses cleanup calls into keyboard handling.  I found that always hung the test shutdown on the curses runner.  (Attached with a python debugger to verify).

http://reviews.llvm.org/D13124

Files:
  test/curses_results.py
  test/dosep.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D13124.35588.patch
Type: text/x-patch
Size: 7942 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150924/4a45b2f7/attachment.bin>


More information about the lldb-commits mailing list