Report eject's failures as logger warnings

If the eject command exits with an error, the output is logged as a WARNING. I don't think it's a good idea to mask those errors.

Closes #354.

Signed-off-by: JoeLametta <[email protected]>

whipper/program/utils.py

@@ -1,4 +1,5 @@
 import os
+import subprocess
 
 import logging
 logger = logging.getLogger(__name__)
@@ -9,15 +10,26 @@ def eject_device(device):
     Eject the given device.
     """
     logger.debug("ejecting device %s", device)
-    os.system('eject %s' % device)
+    try:
+        # `eject device` prints nothing to stdout
+        subprocess.check_output(['eject', device], stderr=subprocess.STDOUT)
+    except subprocess.CalledProcessError as e:
+        logger.warning(e.cmd, 'returned with exit code: ', e.returncode,
+                       e.output)
 
 
 def load_device(device):
     """
     Load the given device.
     """
     logger.debug("loading (eject -t) device %s", device)
-    os.system('eject -t %s' % device)
+    try:
+        # `eject -t device` prints nothing to stdout
+        subprocess.check_output(['eject', '-t', device],
+                                stderr=subprocess.STDOUT)
+    except subprocess.CalledProcessError as e:
+        logger.warning(e.cmd, 'returned with exit code: ', e.returncode,
+                       e.output)
 
 
 def unmount_device(device):