diff options
author | Pavel Labath <labath@google.com> | 2018-01-10 10:18:47 +0000 |
---|---|---|
committer | Pavel Labath <labath@google.com> | 2018-01-10 10:18:47 +0000 |
commit | 698d63a61a51438182127fc4ce56a535d532b182 (patch) | |
tree | 30f8fa133a6297d87d72c083a386ecb7b3efedb7 /lldb/packages/Python/lldbsuite/test | |
parent | 886510f350bd5f4972602263c225098bb2f8aa77 (diff) | |
download | bcm5719-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.py | 34 | ||||
-rw-r--r-- | lldb/packages/Python/lldbsuite/test/crashinfo.c | 64 | ||||
-rw-r--r-- | lldb/packages/Python/lldbsuite/test/dotest.py | 1 | ||||
-rw-r--r-- | lldb/packages/Python/lldbsuite/test/test_result.py | 3 |
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) |