diff options
author | Pavel Labath <pavel@labath.sk> | 2021-02-04 20:53:15 +0100 |
---|---|---|
committer | Pavel Labath <pavel@labath.sk> | 2021-02-11 14:43:53 +0100 |
commit | 7df4eaaa937332c0617aa665080533966e2c98a0 (patch) | |
tree | c685669ee293ecec64028da5a053e109285194e3 /lldb/packages/Python/lldbsuite/test/dotest.py | |
parent | 69f1a7ad82fdcabc8d8e46a80b239c0e772d8e39 (diff) | |
download | llvm-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.py | 6 |
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) |