aboutsummaryrefslogtreecommitdiff
path: root/lldb/packages/Python/lldbsuite/test/dotest.py
diff options
context:
space:
mode:
authorPavel Labath <pavel@labath.sk>2021-02-04 20:53:15 +0100
committerPavel Labath <pavel@labath.sk>2021-02-11 14:43:53 +0100
commit7df4eaaa937332c0617aa665080533966e2c98a0 (patch)
treec685669ee293ecec64028da5a053e109285194e3 /lldb/packages/Python/lldbsuite/test/dotest.py
parent69f1a7ad82fdcabc8d8e46a80b239c0e772d8e39 (diff)
downloadllvm-7df4eaaa937332c0617aa665080533966e2c98a0.zip
llvm-7df4eaaa937332c0617aa665080533966e2c98a0.tar.gz
llvm-7df4eaaa937332c0617aa665080533966e2c98a0.tar.bz2
[lldb/test] Automatically find debug servers to test
Our test configuration logic assumes that the tests can be run either with debugserver or with lldb-server. This is not entirely correct, since lldb server has two "personalities" (platform server and debug server) and debugserver is only a replacement for the latter. A consequence of this is that it's not possible to test the platform behavior of lldb-server on macos, as it is not possible to get a hold of the lldb-server binary. One solution to that would be to duplicate the server configuration logic to be able to specify both executables. However, that seems excessively redundant. A well-behaved lldb should be able to find the debug server on its own, and testing lldb with a different (lldb-|debug)server does not seem very useful (even in the out-of-tree debugserver setup, we copy the server into the build tree to make it appear "real"). Therefore, this patch deletes the configuration altogether and changes the low-level server retrieval functions to be able to both lldb-server and debugserver paths. They do this by consulting the "support executable" directory of the lldb under test. Differential Revision: https://reviews.llvm.org/D96202
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/dotest.py')
-rw-r--r--lldb/packages/Python/lldbsuite/test/dotest.py6
1 files changed, 0 insertions, 6 deletions
diff --git a/lldb/packages/Python/lldbsuite/test/dotest.py b/lldb/packages/Python/lldbsuite/test/dotest.py
index 62508a1..945b3f0 100644
--- a/lldb/packages/Python/lldbsuite/test/dotest.py
+++ b/lldb/packages/Python/lldbsuite/test/dotest.py
@@ -366,12 +366,6 @@ def parseOptionsAndInitTestdirs():
args.executable)
sys.exit(-1)
- if args.server and args.out_of_tree_debugserver:
- logging.warning('Both --server and --out-of-tree-debugserver are set')
-
- if args.server and not args.out_of_tree_debugserver:
- os.environ['LLDB_DEBUGSERVER_PATH'] = args.server
-
if args.excluded:
for excl_file in args.excluded:
parseExclusion(excl_file)