Merge pull request #505 from whipper-team/feature/issue-488-drive-auto-close

Allow configuring whether to auto close the drive's tray
This commit is contained in:
JoeLametta
2020-09-23 19:54:59 +02:00
committed by GitHub
4 changed files with 13 additions and 3 deletions

View File

@@ -47,6 +47,10 @@ Options
| **-e** | **--eject** *never failure success always*
| When to eject disc (default: success)
| **-c** | **--drive-auto-close**
| Whether to auto close the drive's tray before reading a CD
| (default: True)
| **-R** | **--record**
| Record API requests for playback

View File

@@ -92,7 +92,8 @@ class _CD(BaseCommand):
self.device = self.options.device
logger.info('checking device %s', self.device)
utils.load_device(self.device)
if self.options.drive_auto_close is True:
utils.load_device(self.device)
utils.unmount_device(self.device)
# Exit and inform the user if there's no CD in the disk drive
if drive.get_cdrom_drive_status(self.device): # rc == 1 means no disc

View File

@@ -118,7 +118,11 @@ class Whipper(BaseCommand):
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('-c', '--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:

View File

@@ -80,7 +80,8 @@ CD in the AccurateRip database."""
# if necessary, load and unmount
logger.info('checking device %s', device)
utils.load_device(device)
if self.options.drive_auto_close is True:
utils.load_device(device)
utils.unmount_device(device)
# first get the Table Of Contents of the CD