Replace os.system with subprocess

1 files changed, 6 insertions, 2 deletions

diff --git a/src/etoolkit/__main__.py b/src/etoolkit/__main__.py
index 299d75c..a200453 100644
--- a/src/etoolkit/__main__.py
+++ b/src/etoolkit/__main__.py
@@ -28,6 +28,7 @@ import io
 import json
 import logging
 import os
+import subprocess
 import sys
 
 import etoolkit
@@ -324,9 +325,9 @@ def main(inargs=None):
         os.environ.update(env)
 
         if args.spawn:
-            os.system(args.spawn)
+            subprocess.run(args.spawn.split(), check=True)
         else:
-            os.system(os.getenv('SHELL', 'bash'))
+            subprocess.run(os.environ.get('SHELL', 'bash').split(), check=True)
     except KeyboardInterrupt:
         logger.debug('KeyboardInterrupt')
         print(os.linesep)
@@ -334,6 +335,9 @@ def main(inargs=None):
     except etoolkit.EtoolkitInstanceError as e:
         logger.error('EtoolkitInstanceError: %s', e)
         sys.exit(1)
+    except subprocess.CalledProcessError as e:
+        logger.error('Unable to spawn shell process: %s', e)
+        sys.exit(1)
     except Exception as e:
         logger.error('Unexpected exception: %s', e)
         sys.exit(1)