From a2df7f57eefbc78e5f911d82487167851cf8d464 Mon Sep 17 00:00:00 2001
From: R1kaB3rN <100738684+R1kaB3rN@users.noreply.github.com>
Date: Wed, 29 May 2024 16:34:42 -0700
Subject: [PATCH] umu_run: use Popen.wait when waiting for subprocess

---
 umu/umu_run.py | 14 ++++----------
 1 file changed, 4 insertions(+), 10 deletions(-)

diff --git a/umu/umu_run.py b/umu/umu_run.py
index 5977e7bb..cd8dd7df 100755
--- a/umu/umu_run.py
+++ b/umu/umu_run.py
@@ -374,6 +374,7 @@ def run_command(command: list[str]) -> int:
         c_int,
     ] = None
     proc: Popen = None
+    ret: int = 0
     libc: str = get_libc()
 
     if not command:
@@ -406,17 +407,10 @@ def run_command(command: list[str]) -> int:
         start_new_session=True,
         preexec_fn=lambda: prctl(PR_SET_CHILD_SUBREAPER, 1, 0, 0, 0, 0),
     )
+    ret = proc.wait()
+    log.debug("Child %s exited with wait status: %s", proc.pid, ret)
 
-    while True:
-        try:
-            wait_pid, wait_status = os.waitpid(proc.pid, 0)
-            log.debug(
-                "Child %s exited with wait status: %s", wait_pid, wait_status
-            )
-        except ChildProcessError:  # No more children
-            break
-
-    return proc.returncode
+    return ret
 
 
 def main() -> int:  # noqa: D103