summaryrefslogtreecommitdiffstats
path: root/lldb/packages/Python/lldbsuite
diff options
context:
space:
mode:
authorPavel Labath <labath@google.com>2017-10-24 16:07:50 +0000
committerPavel Labath <labath@google.com>2017-10-24 16:07:50 +0000
commite3f6eb1a74cb23dd7872a613cb79fd7e40b2270b (patch)
tree5a7994e965f45f82457a73ebef491d5809afbb3e /lldb/packages/Python/lldbsuite
parent194cb74decd32f8061f358345055b6ae2d5e72e6 (diff)
downloadbcm5719-llvm-e3f6eb1a74cb23dd7872a613cb79fd7e40b2270b.tar.gz
bcm5719-llvm-e3f6eb1a74cb23dd7872a613cb79fd7e40b2270b.zip
Revert "[lldbtests] Handle errors instead of crashing."
The commit breaks the case where you specify just a filename to the compiler. Previously, it would look up the compiler in your path, now it complains that the compiler is not found. One of the lldb buildbots is depending on this. It seems like a nice feature to have, as it means less typing and being able to avoid hard-coding the system compiler path in the bot config. This reverts commit r316393. llvm-svn: 316451
Diffstat (limited to 'lldb/packages/Python/lldbsuite')
-rw-r--r--lldb/packages/Python/lldbsuite/test/dotest.py6
1 files changed, 1 insertions, 5 deletions
diff --git a/lldb/packages/Python/lldbsuite/test/dotest.py b/lldb/packages/Python/lldbsuite/test/dotest.py
index 91f1427136c..7d739acba95 100644
--- a/lldb/packages/Python/lldbsuite/test/dotest.py
+++ b/lldb/packages/Python/lldbsuite/test/dotest.py
@@ -50,11 +50,7 @@ from ..support import seven
def is_exe(fpath):
- """Returns true if fpath is an executable.
- Exits with an error code if the specified path is invalid"""
- if not os.path.exists(fpath):
- print(fpath + " is not a valid path, exiting")
- sys.exit(-1)
+ """Returns true if fpath is an executable."""
return os.path.isfile(fpath) and os.access(fpath, os.X_OK)
OpenPOWER on IntegriCloud