[Lldb-commits] [PATCH] D52498: [lldb-mi] Fix bugs in target-select-so-path.test

Alexander Polyakov via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Sep 25 08:01:07 PDT 2018


apolyakov updated this revision to Diff 166919.
apolyakov added a comment.

Removed the shebang line since we call the script via `python ...` so that line doesn't matter.


https://reviews.llvm.org/D52498

Files:
  lit/tools/lldb-mi/target/inputs/target-select-so-path.py


Index: lit/tools/lldb-mi/target/inputs/target-select-so-path.py
===================================================================
--- lit/tools/lldb-mi/target/inputs/target-select-so-path.py
+++ lit/tools/lldb-mi/target/inputs/target-select-so-path.py
@@ -1,5 +1,3 @@
-#!/usr/bin/env python2
-
 import os
 import sys
 import subprocess
@@ -9,22 +7,26 @@
 hostname = 'localhost'
 
 (r, w) = os.pipe()
+kwargs = {}
+if sys.version_info >= (3,0):
+    kwargs['pass_fds'] = [w]
+
 args = sys.argv
 # Get debugserver, lldb-mi and FileCheck executables' paths with arguments.
 debugserver = ' '.join([args[1], '--pipe', str(w), hostname + ':0'])
 lldbmi = args[2]
 test_file = args[3]
 filecheck = 'FileCheck ' + test_file
 
 # Run debugserver, lldb-mi and FileCheck.
-debugserver_proc = subprocess.Popen(debugserver.split())
+debugserver_proc = subprocess.Popen(debugserver.split(), **kwargs)
 lldbmi_proc = subprocess.Popen(lldbmi, stdin=subprocess.PIPE,
                                stdout=subprocess.PIPE, shell=True)
 filecheck_proc = subprocess.Popen(filecheck, stdin=subprocess.PIPE,
                                   shell=True)
 
 timeout_sec = 30
-timer = Timer(timeout_sec, lldbmi_proc.kill)
+timer = Timer(timeout_sec, exit, [filecheck_proc.returncode])
 try:
     timer.start()
 
@@ -37,9 +39,10 @@
     with open(test_file, 'r') as f:
         # Replace '$PORT' with a free port number and pass
         # test's content to lldb-mi.
-        lldbmi_proc.stdin.write(f.read().replace('$PORT', port))
+        lldbmi_proc.stdin.write(f.read().replace('$PORT', port).encode('utf-8'))
         out, err = lldbmi_proc.communicate()
         filecheck_proc.stdin.write(out)
+        filecheck_proc.communicate()
 finally:
     timer.cancel()
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52498.166919.patch
Type: text/x-patch
Size: 1752 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20180925/f2489023/attachment-0001.bin>


More information about the lldb-commits mailing list