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

[Todd Fiala via lldb-commits]

tfiala updated this revision to Diff 35814.
tfiala added a comment.

Minor refactor, moving new components to test/test_runner/lib.

Will be adding test runner tests to test/test_runner/test and will make sure the normal test runner doesn't try to run them.  (These will be test infrastructure tests, i.e. tests for the test infrastructure, not tests that run on them).

This is vs. llvm.org lldb trunk r248676.

Note I've also now had positive timeout confirmation on OS X.  As I suspected, it is less convenient there to have the soft kill do a SIGQUIT, since on OS X this brings up the crash log generator dialog where now user input is required (and very bad if the tests are running in a session that doesn't have access to a window server).  I'll be working in a --no-core-on-timeout and --core-on-timeout option, and have it default to "no" on OS X and "yes" everywhere else.  When no cores are desired, we'll generate a SIGTERM instead of a SIGQUIT on the soft terminate request.


http://reviews.llvm.org/D13124

Files:
  test/dosep.py
  test/test_runner/README.txt
  test/test_runner/lib/lldb_utils.py
  test/test_runner/lib/process_control.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D13124.35814.patch
Type: text/x-patch
Size: 28686 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150926/8f1ab789/attachment-0001.bin>