update logging_subprocess function

slibby · web-flow · commit 4f4785085de0 · 2017-12-11T09:25:49.000-07:00
1. added newline for return
2. added one-time warning (once per subprocess)
diff --git 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],
                                       [],
                                       [],
