]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
[3.13] gh-131038: Use text=True in subprocesses in test_perf_profiler (GH-137117...
authorMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Sat, 26 Jul 2025 17:24:32 +0000 (19:24 +0200)
committerGitHub <noreply@github.com>
Sat, 26 Jul 2025 17:24:32 +0000 (17:24 +0000)
Lib/test/test_perf_profiler.py

index d628acb165e6ae8df9310233f2e07d26893c9d47..0af8670c9180c8168de12e626801f840b5da0dc7 100644 (file)
@@ -312,6 +312,7 @@ def run_perf(cwd, *args, use_jit=False, **env_vars):
         stdout=subprocess.PIPE,
         stderr=subprocess.PIPE,
         env=env,
+        text=True,
     )
     if proc.returncode:
         print(proc.stderr, file=sys.stderr)
@@ -321,10 +322,10 @@ def run_perf(cwd, *args, use_jit=False, **env_vars):
         jit_output_file = cwd + "/jit_output.dump"
         command = ("perf", "inject", "-j", "-i", output_file, "-o", jit_output_file)
         proc = subprocess.run(
-            command, stderr=subprocess.PIPE, stdout=subprocess.PIPE, env=env
+            command, stderr=subprocess.PIPE, stdout=subprocess.PIPE, env=env, text=True
         )
         if proc.returncode:
-            print(proc.stderr)
+            print(proc.stderr, file=sys.stderr)
             raise ValueError(f"Perf failed with return code {proc.returncode}")
         # Copy the jit_output_file to the output_file
         os.rename(jit_output_file, output_file)
@@ -336,10 +337,9 @@ def run_perf(cwd, *args, use_jit=False, **env_vars):
         stderr=subprocess.PIPE,
         env=env,
         check=True,
+        text=True,
     )
-    return proc.stdout.decode("utf-8", "replace"), proc.stderr.decode(
-        "utf-8", "replace"
-    )
+    return proc.stdout, proc.stderr
 
 
 class TestPerfProfilerMixin: