Using start_new_session instead of preexec_fn in subprocess.Popen(). …

…Comment update

PILOTVERSION

@@ -1 +1 @@
-3.9.2.1
+3.9.2.2

pilot/control/payloads/generic.py

@@ -842,7 +842,7 @@ def run(self) -> tuple[int, str]:  # noqa: C901
                         if stdout and stderr
                         else "General payload setup verification error (check setup logs)"
                     )
-                    # check for special errors in thw output
+                    # check for special errors in the output
                     exit_code = errors.resolve_transform_error(exit_code, diagnostics)
                     diagnostics = errors.format_diagnostics(exit_code, diagnostics)
                     return exit_code, diagnostics

pilot/util/constants.py

@@ -28,7 +28,7 @@
 RELEASE = '3'   # released number should be fixed at 3 for Pilot 3
 VERSION = '9'   # version number is '1' for first release, '0' until then, increased for bigger updates
 REVISION = '2'  # revision number should be reset to '0' for every new version release, increased for small updates
-BUILD = '1'     # build number should be reset to '1' for every new development cycle
+BUILD = '2'     # build number should be reset to '1' for every new development cycle
 
 SUCCESS = 0
 FAILURE = 1

pilot/util/container.py

@@ -100,7 +100,8 @@ def execute(executable: Any, **kwargs: dict) -> Any:  # noqa: C901
                                    stdout=kwargs.get('stdout', subprocess.PIPE),
                                    stderr=kwargs.get('stderr', subprocess.PIPE),
                                    cwd=kwargs.get('cwd', getcwd()),
-                                   preexec_fn=os.setsid,    # setpgrp
+                                   start_new_session=True,  # alternative to use os.setsid
+                                   # preexec_fn=os.setsid,    # setpgrp
                                    encoding='utf-8',
                                    errors='replace')
         if kwargs.get('returnproc', False):