Merge pull request #75 from slibby/slibby-patch-windows

update check_io() to allow scripts to run on Windows
This commit is contained in:
Jose Diaz-Gonzalez
2017-12-11 11:45:58 -05:00
committed by GitHub

View File

@@ -69,11 +69,15 @@ def logging_subprocess(popenargs,
""" """
child = subprocess.Popen(popenargs, stdout=subprocess.PIPE, child = subprocess.Popen(popenargs, stdout=subprocess.PIPE,
stderr=subprocess.PIPE, **kwargs) 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, log_level = {child.stdout: stdout_log_level,
child.stderr: stderr_log_level} child.stderr: stderr_log_level}
def check_io(): def check_io():
if sys.platform == 'win32':
return
ready_to_read = select.select([child.stdout, child.stderr], ready_to_read = select.select([child.stdout, child.stderr],
[], [],
[], [],