[SCM] morituri/master: handle not having cdparanoia on rip offset find

js at users.alioth.debian.org js at users.alioth.debian.org
Sun Oct 19 20:10:08 UTC 2014


The following commit has been merged in the master branch:
commit 28926e3fba5b88886631e73a472476558b0c6998
Author: Thomas Vander Stichele <thomas (at) apestaart (dot) org>
Date:   Tue Jul 30 10:59:28 2013 +0200

    handle not having cdparanoia on rip offset find

diff --git a/morituri/rip/offset.py b/morituri/rip/offset.py
index 646d46e..965c9f0 100644
--- a/morituri/rip/offset.py
+++ b/morituri/rip/offset.py
@@ -26,7 +26,7 @@ import tempfile
 import gobject
 gobject.threads_init()
 
-from morituri.common import logcommand, accurip, drive, program
+from morituri.common import logcommand, accurip, drive, program, common
 from morituri.common import task as ctask
 from morituri.program import cdrdao, cdparanoia
 
@@ -151,11 +151,18 @@ CD in the AccurateRip database."""
             try:
                 archecksum = self._arcs(runner, table, 1, offset)
             except task.TaskException, e:
+
+                # let MissingDependency fall through
+                if isinstance(e.exception,
+                    common.MissingDependencyException):
+                    raise e
+
                 if isinstance(e.exception, cdparanoia.FileSizeError):
                     self.stdout.write(
                         'WARNING: cannot rip with offset %d...\n' % offset)
                     continue
-                self.warning("Unknown exception for offset %d: %r" % (
+
+                self.warning("Unknown task exception for offset %d: %r" % (
                     offset, e))
                 self.stdout.write(
                     'WARNING: cannot rip with offset %d...\n' % offset)

-- 
morituri packaging



More information about the pkg-multimedia-commits mailing list