Bug fix for core not working in python 3 (#62)

Author: rane-rajasi

src/core/src/bootstrap/EnvLayer.py

@@ -96,12 +96,18 @@ def __run_command_output_raw(self, cmd, no_output, chk_err=True):
         Reports exceptions to Error if chk_err parameter is True
         """
 
-        def check_output(no_output, *popenargs, **kwargs):
+        def check_output(*popenargs, **kwargs):
             """
             Backport from subprocess module from python 2.7
             """
             if 'stdout' in kwargs:
                 raise ValueError('stdout argument not allowed, it will be overridden.')
+
+            no_output = False
+            if type(popenargs[0]) is bool:
+                no_output = popenargs[0]
+                popenargs = popenargs[1:]
+
             if no_output is True:
                 out_file = None
             else:

src/core/src/local_loggers/StdOutFileMirror.py

@@ -18,6 +18,7 @@
 If the log file language is set to 'Python' in Notepad++, with code as implemented below, useful collapsibility is obtained."""
 import sys
 
+
 class StdOutFileMirror(object):
     """Mirrors all terminal output to a local file"""
 

src/extension/src/ProcessHandler.py

@@ -114,12 +114,18 @@ def __run_command_output_raw(self, cmd, no_output, chk_err=True):
         Reports exceptions to Error if chk_err parameter is True
         """
 
-        def check_output(no_output, *popenargs, **kwargs):
+        def check_output(*popenargs, **kwargs):
             """
             Backport from subprocess module from python 2.7
             """
             if 'stdout' in kwargs:
                 raise ValueError('stdout argument not allowed, it will be overridden.')
+
+            no_output = False
+            if type(popenargs[0]) is bool:
+                no_output = popenargs[0]
+                popenargs = popenargs[1:]
+
             if no_output is True:
                 out_file = None
             else: