This is an archive of the discontinued LLVM Phabricator instance.

Fix _LocalProcess.terminate on Windows
ClosedPublic

Authored by amccarth on Jul 6 2015, 3:12 PM.

Details

Reviewers
labath
Summary

Some of the signals terminate() was trying to send aren't available on Windows, specifically SIGHUP and SIGCONT. This caused lldbtest.py to crash and leave the processes running. This fix should be behavior-preserving on Mac and Linux.

Note that Windows defines SIGINT, but claims it's not a valid signal to send, thus the try/except.

This fix makes it once again possible to run all LLDB tests on Windows, but I'm still investigating why ninja check-lldb no longer works.

Warning: I'm a Python newbie.

Diff Detail

Event Timeline

amccarth updated this revision to Diff 29130.Jul 6 2015, 3:12 PM
amccarth retitled this revision from to Fix _LocalProcess.terminate on Windows.
amccarth updated this object.
amccarth added a reviewer: labath.
amccarth added a subscriber: lldb-commits.
labath accepted this revision.Jul 7 2015, 1:56 AM
labath edited edge metadata.

lgtm

This revision is now accepted and ready to land.Jul 7 2015, 1:56 AM
labath closed this revision.Jul 8 2015, 1:40 AM