Skip to content

Commit

Permalink
Merge pull request #506 from whipper-team/ux/issue-495-cdparanoia-war…
Browse files Browse the repository at this point in the history
…nings

Add checks and warnings for (known) cdparanoia's upstream bugs
  • Loading branch information
JoeLametta authored Sep 23, 2020
2 parents b754b2b + 07bd034 commit 302bc22
Showing 1 changed file with 18 additions and 0 deletions.
18 changes: 18 additions & 0 deletions whipper/program/cdparanoia.py
Original file line number Diff line number Diff line change
Expand Up @@ -283,6 +283,24 @@ def start(self, runner):
stopTrack, common.framesToHMSF(stopOffset)),
self.path])
logger.debug('running %s', (" ".join(argv), ))
if self._offset > 587:
logger.warning(
"because of a cd-paranoia upstream bug whipper may fail to "
"work correctly when using offset values > 587 (current "
"value: %d) and print warnings like this: 'file size 0 did "
"not match expected size'. For more details please check the "
"following issues: "
"https://github.com/whipper-team/whipper/issues/234 and "
"https://github.com/rocky/libcdio-paranoia/issues/14",
self._offset
)
if stopTrack == 99:
logger.warning(
"because of a cd-paranoia upstream bug whipper may fail to "
"rip the last track of a CD when it has got 99 of them. "
"For more details please check the following issue: "
"https://github.com/whipper-team/whipper/issues/302"
)
try:
self._popen = asyncsub.Popen(argv,
bufsize=bufsize,
Expand Down

0 comments on commit 302bc22

Please sign in to comment.