tools: make pty_helper.py python3-compatible

Fixes: #29166 PR-URL: #29167 Reviewed-By: Sam Roberts <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Rich Trott <[email protected]>
nodejs · Aug 20, 2019 · acdc21b · acdc21b
1 parent 5c5ef4e
commit acdc21b
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/test/pseudo-tty/pty_helper.py b/test/pseudo-tty/pty_helper.py
@@ -25,8 +25,12 @@ def pipe(sfd, dfd):
   if dfd == STDOUT:
     # Work around platform quirks. Some platforms echo ^D as \x04
     # (AIX, BSDs) and some don't (Linux).
-    filt = lambda c: ord(c) > 31 or c in '\t\n\r\f'
+    #
+    # The comparison against b' '[0] is because |data| is
+    # a string in python2 but a bytes object in python3.
+    filt = lambda c: c >= b' '[0] or c in b'\t\n\r\f'
     data = filter(filt, data)
+    data = bytes(data)
 
   while data:
     try: