From 62baabc31e86faca0cb554816a83ad38996272f7 Mon Sep 17 00:00:00 2001
From: John Jackson <39013067+johnjhacking@users.noreply.github.com>
Date: Mon, 7 Jun 2021 13:52:20 -0600
Subject: [PATCH] Change call function to Popen

The call function can deadlock if the child process prints larger output. Use Popen with the communicate() method with you need pipes. If you switch to Popen, you'll get cleaner debugging.
---
 fetch-macOS.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/fetch-macOS.py b/fetch-macOS.py
index f4d7e36..b540db9 100755
--- a/fetch-macOS.py
+++ b/fetch-macOS.py
@@ -87,8 +87,8 @@ class ReplicationError(Exception):
 
 
 def cmd_exists(cmd):
-    return subprocess.call("type " + cmd, shell=True,
-                           stdout=subprocess.PIPE, stderr=subprocess.PIPE) == 0
+    proc subprocess.Popen("type " + cmd, shell=True,
+                           stdout=subprocess.PIPE, stderr=subprocess.PIPE)
 
 
 def replicate_url(full_url,