Allow configuring whether to auto close the drive's tray

Untested. Fixes #488. Signed-off-by: JoeLametta <[email protected]>
whipper-team · Sep 17, 2020 · 06f1eec · 06f1eec
1 parent 3acc3ff
commit 06f1eec
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 3 deletions.
diff --git a/man/whipper.rst b/man/whipper.rst
@@ -47,6 +47,10 @@ Options
 | **-e** | **--eject**  *never failure success always*
 |     When to eject disc (default: success)
 
+| **--drive-auto-close**
+|     Whether to auto close the drive's tray before reading a CD
+|     (default: True)
+
 | **-R** | **--record**
 |     Record API requests for playback
 

diff --git a/whipper/command/cd.py b/whipper/command/cd.py
@@ -92,7 +92,8 @@ def do(self):
         self.device = self.options.device
         logger.info('checking device %s', self.device)
 
-        utils.load_device(self.device)
+        if self.options.drive_auto_close:
+            utils.load_device(self.device)
         utils.unmount_device(self.device)
 
         # first, read the normal TOC, which is fast

diff --git a/whipper/command/main.py b/whipper/command/main.py
@@ -118,7 +118,11 @@ def add_arguments(self):
                                  default="success",
                                  choices=('never', 'failure',
                                           'success', 'always'),
-                                 help="when to eject disc (default: success)")
+                                 help="when to eject disc (default: success)"),
+        self.parser.add_argument('--drive-auto-close', action="store",
+                                 dest="drive_auto_close", default=True,
+                                 help="whether to auto close the drive's "
+                                 "tray before reading a CD (default: True)")
 
     def handle_arguments(self):
         if self.options.help:

diff --git a/whipper/command/offset.py b/whipper/command/offset.py
@@ -80,7 +80,8 @@ def do(self):
         # if necessary, load and unmount
         logger.info('checking device %s', device)
 
-        utils.load_device(device)
+        if self.options.drive_auto_close:
+            utils.load_device(device)
         utils.unmount_device(device)
 
         # first get the Table Of Contents of the CD