From 4f4785085de05bf86fd26b2cbedd3123ec541288 Mon Sep 17 00:00:00 2001
From: Sam Libby <slibby@esri.com>
Date: Mon, 11 Dec 2017 09:25:49 -0700
Subject: [PATCH] update logging_subprocess function

1. added newline for return
2. added one-time warning (once per subprocess)
---
 bin/github-backup | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/bin/github-backup b/bin/github-backup
index 9aa920d..1f4c73e 100755
--- a/bin/github-backup
+++ b/bin/github-backup
@@ -69,12 +69,15 @@ def logging_subprocess(popenargs,
     """
     child = subprocess.Popen(popenargs, stdout=subprocess.PIPE,
                              stderr=subprocess.PIPE, **kwargs)
+    if sys.platform == 'win32':
+        log_info("Windows operating system detected - no subprocess logging will be returned")
 
     log_level = {child.stdout: stdout_log_level,
                  child.stderr: stderr_log_level}
 
     def check_io():
-        if sys.platform == 'win32': return
+        if sys.platform == 'win32':
+            return
         ready_to_read = select.select([child.stdout, child.stderr],
                                       [],
                                       [],