[Lldb-commits] [lldb] r229357 - Fix TestRegisters on remote target.
Chaoren Lin
chaorenl at google.com
Sun Feb 15 22:17:51 PST 2015
Author: chaoren
Date: Mon Feb 16 00:17:51 2015
New Revision: 229357
URL: http://llvm.org/viewvc/llvm-project?rev=229357&view=rev
Log:
Fix TestRegisters on remote target.
Summary:
Using spawnSubprocess instead of forkSubprocess, so that the subprocess
spawns on the target and not the host.
Requires http://reviews.llvm.org/D7660 for cleanup.
Test Plan: TestRegisters.py passing.
Reviewers: ovyalov, vharron, clayborg
Subscribers: lldb-commits
Differential Revision: http://reviews.llvm.org/D7661
Modified:
lldb/trunk/test/functionalities/register/TestRegisters.py
Modified: lldb/trunk/test/functionalities/register/TestRegisters.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/test/functionalities/register/TestRegisters.py?rev=229357&r1=229356&r2=229357&view=diff
==============================================================================
--- lldb/trunk/test/functionalities/register/TestRegisters.py (original)
+++ lldb/trunk/test/functionalities/register/TestRegisters.py Mon Feb 16 00:17:51 2015
@@ -252,12 +252,7 @@ class RegisterCommandsTestCase(TestBase)
exe = os.path.join(os.getcwd(), "a.out")
# Spawn a new process
- pid = 0
- if sys.platform.startswith('linux'):
- pid = self.forkSubprocess(exe, ['wait_for_attach'])
- else:
- proc = self.spawnSubprocess(exe, ['wait_for_attach'])
- pid = proc.pid
+ pid = self.spawnSubprocess(exe, ['wait_for_attach']).pid
self.addTearDownHook(self.cleanupSubprocesses)
if self.TraceOn():
More information about the lldb-commits
mailing list