Add option to keep unverified tracks. Fixes #612.

Add a -u/--keep-unverified option that keeps the most recent attempt
of ripping a track that failed verification.

The logging might be inconsistent in that the unverifiable track is
mentioned as being skipped in some log messages.

Signed-off-by: leper <leper4@protonmail.com>

Author: leper

man/whipper-cd-rip.rst

@@ -73,6 +73,10 @@ Options
 |     continue ripping further tracks instead of giving up if a track can't be
 |     ripped
 
+| **-u** | **--keep-unverified**
+|     keep unverified tracks instead of deleting the data if a track can't be
+|     verified
+
 Template schemes
 ================
 

whipper/command/cd.py

@@ -326,6 +326,11 @@ def add_arguments(self):
                                  help="continue ripping further tracks "
                                  "instead of giving up if a track "
                                  "can't be ripped")
+        self.parser.add_argument('-u', '--keep-unverified',
+                                 action='store_true',
+                                 help="keep unverified (partial) results "
+                                 "instead of deleleting the data if a track "
+                                 "can't be verified")
 
     def handle_arguments(self):
         self.options.output_directory = os.path.expanduser(
@@ -481,7 +486,8 @@ def _ripIfNotRipped(number):
                                                   number,
                                                   len(self.itable.tracks),
                                                   extra),
-                                              coverArtPath=self.coverArtPath)
+                                              coverArtPath=self.coverArtPath,
+                                              keep=self.options.keep_unverified)
                         break
                     # FIXME: catching too general exception (Exception)
                     except Exception as e:
@@ -492,7 +498,15 @@ def _ripIfNotRipped(number):
                     tries -= 1
                     logger.critical('giving up on track %d after %d times',
                                     number, tries)
-                    if self.options.keep_going:
+                    if self.options.keep_unverified and not number == 0:
+                        logger.warning("track %d failed to rip. keeping unverified file.", number)
+                        logger.debug("adding %s to skipped_tracks",
+                                     trackResult)
+                        self.skipped_tracks.append(trackResult)
+                        logger.debug("skipped_tracks = %s",
+                                     self.skipped_tracks)
+                        trackResult.skipped = True
+                    elif self.options.keep_going:
                         logger.warning("track %d failed to rip.", number)
                         logger.debug("adding %s to skipped_tracks",
                                      trackResult)

whipper/common/program.py

@@ -562,7 +562,7 @@ def verifyTrack(runner, trackResult):
         return ret
 
     def ripTrack(self, runner, trackResult, offset, device, taglist,
-                 overread, what=None, coverArtPath=None):
+                 overread, what=None, coverArtPath=None, keep=False):
         """
         Rip and store a track of the disc.
 
@@ -606,7 +606,8 @@ def ripTrack(self, runner, trackResult, offset, device, taglist,
                                            device=device,
                                            taglist=taglist,
                                            what=what,
-                                           coverArtPath=coverArtPath)
+                                           coverArtPath=coverArtPath,
+                                           keep=keep)
 
         runner.run(t)
 

whipper/program/cdparanoia.py

@@ -439,8 +439,10 @@ class ReadVerifyTrackTask(task.MultiSeparateTask):
     _tmpwavpath = None
     _tmppath = None
 
+    _keep = False
+
     def __init__(self, path, table, start, stop, overread, offset=0,
-                 device=None, taglist=None, what="track", coverArtPath=None):
+                 device=None, taglist=None, what="track", coverArtPath=None, keep=False):
         """
         Init ReadVerifyTrackTask.
 
@@ -471,6 +473,8 @@ def __init__(self, path, table, start, stop, overread, offset=0,
         os.close(fd)
         self._tmpwavpath = tmppath
 
+        self._keep = keep
+
         from whipper.common import checksum
 
         self.tasks = []
@@ -547,9 +551,11 @@ def stop(self):
                 # delete the unencoded file
                 os.unlink(self._tmpwavpath)
 
-                if not self.exception:
+                if not self.exception or self._keep:
                     try:
                         logger.debug('moving to final path %r', self.path)
+                        if self.exception:
+                            logger.debug('keeping unverified result')
                         shutil.move(self._tmppath, self.path)
                     # FIXME: catching too general exception (Exception)
                     except Exception as e: