summaryrefslogtreecommitdiffstats
path: root/lldb/packages/Python/lldbsuite/test
diff options
context:
space:
mode:
authorPavel Labath <labath@google.com>2018-01-10 10:18:47 +0000
committerPavel Labath <labath@google.com>2018-01-10 10:18:47 +0000
commit698d63a61a51438182127fc4ce56a535d532b182 (patch)
tree30f8fa133a6297d87d72c083a386ecb7b3efedb7 /lldb/packages/Python/lldbsuite/test
parent886510f350bd5f4972602263c225098bb2f8aa77 (diff)
downloadbcm5719-llvm-698d63a61a51438182127fc4ce56a535d532b182.tar.gz
bcm5719-llvm-698d63a61a51438182127fc4ce56a535d532b182.zip
[dotest] Remove crashinfo hook
Summary: This used to be important when all tests were run in a single process, but that has no longer been the case for a while. Furthermore, this hook fails to build on new mac versions for several people, and it's not clear whether fixing it is worth the effort. Reviewers: jingham, clayborg, davide Subscribers: lldb-commits Differential Revision: https://reviews.llvm.org/D41871 llvm-svn: 322167
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test')
-rw-r--r--lldb/packages/Python/lldbsuite/test/configuration.py34
-rw-r--r--lldb/packages/Python/lldbsuite/test/crashinfo.c64
-rw-r--r--lldb/packages/Python/lldbsuite/test/dotest.py1
-rw-r--r--lldb/packages/Python/lldbsuite/test/test_result.py3
4 files changed, 0 insertions, 102 deletions
diff --git a/lldb/packages/Python/lldbsuite/test/configuration.py b/lldb/packages/Python/lldbsuite/test/configuration.py
index a49948c0205..25fb1b1b474 100644
--- a/lldb/packages/Python/lldbsuite/test/configuration.py
+++ b/lldb/packages/Python/lldbsuite/test/configuration.py
@@ -24,40 +24,6 @@ import unittest2
import lldbsuite
-def __setCrashInfoHook_Mac(text):
- from . import crashinfo
- crashinfo.setCrashReporterDescription(text)
-
-
-def setupCrashInfoHook():
- if platform.system() == "Darwin":
- from . import lock
- test_dir = os.environ['LLDB_TEST']
- if not test_dir or not os.path.exists(test_dir):
- return
- dylib_lock = os.path.join(test_dir, "crashinfo.lock")
- dylib_src = os.path.join(test_dir, "crashinfo.c")
- dylib_dst = os.path.join(test_dir, "crashinfo.so")
- try:
- compile_lock = lock.Lock(dylib_lock)
- compile_lock.acquire()
- if not os.path.isfile(dylib_dst) or os.path.getmtime(
- dylib_dst) < os.path.getmtime(dylib_src):
- # we need to compile
- cmd = "SDKROOT= xcrun clang %s -o %s -framework Python -Xlinker -dylib" % (
- dylib_src, dylib_dst)
- if subprocess.call(
- cmd, shell=True) != 0 or not os.path.isfile(dylib_dst):
- raise Exception('command failed: "{}"'.format(cmd))
- finally:
- compile_lock.release()
- del compile_lock
-
- setCrashInfoHook = __setCrashInfoHook_Mac
-
- else:
- pass
-
# The test suite.
suite = unittest2.TestSuite()
diff --git a/lldb/packages/Python/lldbsuite/test/crashinfo.c b/lldb/packages/Python/lldbsuite/test/crashinfo.c
deleted file mode 100644
index 19013468bb2..00000000000
--- a/lldb/packages/Python/lldbsuite/test/crashinfo.c
+++ /dev/null
@@ -1,64 +0,0 @@
-/******************************************************************************
- The LLVM Compiler Infrastructure
-
- This file is distributed under the University of Illinois Open Source
- License. See LICENSE.TXT for details.
- ******************************************************************************
-
-* This C file vends a simple interface to set the Application Specific Info
-* on Mac OS X through Python. To use, compile as a dylib, import crashinfo
-* and call crashinfo.setCrashReporterDescription("hello world")
-* The testCrashReporterDescription() API is simply there to let you test that this
-* is doing what it is intended to do without having to actually cons up a crash
-******************************************************************************/
-
-#include <Python/Python.h>
-#include <stdlib.h>
-#include <string.h>
-#include <stdio.h>
-
-void *__crashreporter_info__ = NULL;
-
-asm(".desc ___crashreporter_info__, 0x10");
-
-static PyObject* setCrashReporterDescription(PyObject* self, PyObject* string)
-{
- if (__crashreporter_info__)
- {
- free(__crashreporter_info__);
- __crashreporter_info__ = NULL;
- }
-
- if (string && PyString_Check(string))
- {
- Py_ssize_t size = PyString_Size(string);
- char* data = PyString_AsString(string);
- if (size > 0 && data)
- {
- ++size; // Include the NULL terminateor in allocation and memcpy()
- __crashreporter_info__ = malloc(size);
- memcpy(__crashreporter_info__, data, size);
- return Py_True;
- }
- }
- return Py_False;
-}
-
-static PyObject* testCrashReporterDescription(PyObject*self, PyObject* arg)
-{
- int* ptr = 0;
- *ptr = 1;
- return Py_None;
-}
-
-static PyMethodDef crashinfo_methods[] = {
- {"setCrashReporterDescription", setCrashReporterDescription, METH_O},
- {"testCrashReporterDescription", testCrashReporterDescription, METH_O},
- {NULL, NULL}
-};
-
-void initcrashinfo()
-{
- (void) Py_InitModule("crashinfo", crashinfo_methods);
-}
-
diff --git a/lldb/packages/Python/lldbsuite/test/dotest.py b/lldb/packages/Python/lldbsuite/test/dotest.py
index 7d739acba95..c7c60a748e4 100644
--- a/lldb/packages/Python/lldbsuite/test/dotest.py
+++ b/lldb/packages/Python/lldbsuite/test/dotest.py
@@ -1117,7 +1117,6 @@ def run_suite():
signal.signal(signal.SIGINT, signal.SIG_IGN)
setupSysPath()
- configuration.setupCrashInfoHook()
#
# If '-l' is specified, do not skip the long running tests.
diff --git a/lldb/packages/Python/lldbsuite/test/test_result.py b/lldb/packages/Python/lldbsuite/test/test_result.py
index 3f5e556040b..3e3460b49c5 100644
--- a/lldb/packages/Python/lldbsuite/test/test_result.py
+++ b/lldb/packages/Python/lldbsuite/test/test_result.py
@@ -171,9 +171,6 @@ class LLDBTestResult(unittest2.TextTestResult):
(str(test), inspect.getfile(
test.__class__)))
self.counter += 1
- # if self.counter == 4:
- # import crashinfo
- # crashinfo.testCrashReporterDescription(None)
test.test_number = self.counter
if self.showAll:
self.stream.write(self.fmt % self.counter)
OpenPOWER on IntegriCloud