Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Can't rip last track on a CD with 99 tracks #302

Open
ghost opened this issue Oct 4, 2018 · 6 comments
Open

Can't rip last track on a CD with 99 tracks #302

ghost opened this issue Oct 4, 2018 · 6 comments
Assignees
Labels
Accepted Accepted issue on our roadmap Bug Generic bug: can be used together with more specific labels Needed: discussion More discussion needed before anything can be done (or still no agreement has been reached) Needed: patch A pull request is required Priority: medium Medium priority Stretch Optional goal for the current sprint (may not be delivered) Upstream Bug The issue is a result of an upstream bug
Milestone

Comments

@ghost
Copy link

ghost commented Oct 4, 2018

I can't rip the last track of a CD with 99 tracks. I tried the following command:

$ WHIPPER_DEBUG=DEBUG WHIPPER_LOGFILE=whipper.log whipper cd rip -R 5cdfa69d-7b0c-3579-af99-901b03fe33ea

Everything went well until the last track:

...
Ripping track 99 of 99: 99 [unknown].flac
Ripping track 99 of 99 (try 2): 99 [unknown].flac
Ripping track 99 of 99 (try 3): 99 [unknown].flac
Ripping track 99 of 99 (try 4): 99 [unknown].flac
Ripping track 99 of 99 (try 5): 99 [unknown].flac
track can't be ripped. Rip attempts number is equal to 'MAX_TRIES'

whipper.log.gz

Some more info:

$ whipper drive list
drive: /dev/cdrom, vendor: PLEXTOR , model: DVDR   PX-830SA , release: 1.01
       Configured read offset: 96
       Can defeat audio cache: True

I'm using media-sound/whipper-0.7.0 on a Gentoo Linux system.

@JoeLametta
Copy link
Collaborator

Thanks for reporting this issue!
Could you post the output of the following command?

libcdio-paranoia --stderr-progress --sample-offset=96 --force-cdrom-device /dev/sr0 99[00:00:00.00]-99[00:01:39.14] "$HOME/track_99.wav"

If that fails you may need to: umount /dev/sr0 first.

@JoeLametta JoeLametta added the Bug Generic bug: can be used together with more specific labels label Oct 6, 2018
@ghost
Copy link
Author

ghost commented Oct 6, 2018

It immediately fails with the following error message:

$ libcdio-paranoia --stderr-progress --sample-offset=96 --force-cdrom-device /dev/sr0 99[00:00:00.00]-99[00:01:39.14] $HOME/track_99.wav
Sending all callback output to stderr for wrapper script
cdparanoia III release 10.2 libcdio 2.0.0 x86_64-pc-linux-gnu
(C) 2001 Monty <[email protected]> and Xiphophorus
(C) 2004, 2005, 2008 Rocky Bernstein <[email protected]>
(C) 2014 Robert Kausch <[email protected]>

Report bugs to [email protected]

Time/sector offset goes beyond end of specified track.

@JoeLametta
Copy link
Collaborator

Uhm... given the error message the issue seems similar to the one reported in #234 (and libcdio/libcdio-paranoia#14).

Could you please install Xiph's cdparanoia and post the output of the following command?

cdparanoia --stderr-progress --sample-offset=96 --force-cdrom-device /dev/sr0 99[00:00:00.00]-99[00:01:39.14] "$HOME/track_99_cdparanoia.wav"

Cheers,
Joe

@ghost
Copy link
Author

ghost commented Oct 8, 2018

cdparanoia.log.gz

I used the following command:

cdparanoia-paranoia --stderr-progress --sample-offset=96 --force-cdrom-device /dev/sr0 99[00:00:00.00]-99[00:01:39.14] ~/track_99_cdparanoia.wav > cdparanoia.log 2>&1

On my system cdparanoia-paranoia is Xiph's one, while libcdio-paranoia is the updated one.

Some info about the output track:

$ du -b track_99_cdparanoia.wav 
17498924	track_99_cdparanoia.wav
$ mediainfo track_99_cdparanoia.wav 
General
Complete name                            : track_99_cdparanoia.wav
Format                                   : Wave
File size                                : 16.7 MiB
Duration                                 : 1 min 39 s
Overall bit rate mode                    : Constant
Overall bit rate                         : 1 411 kb/s

Audio
Format                                   : PCM
Format settings                          : Little / Signed
Codec ID                                 : 1
Duration                                 : 1 min 39 s
Bit rate mode                            : Constant
Bit rate                                 : 1 411.2 kb/s
Channel(s)                               : 2 channels
Sampling rate                            : 44.1 kHz
Bit depth                                : 16 bits
Stream size                              : 16.7 MiB (100%)

@JoeLametta
Copy link
Collaborator

All right, that confirms my suspicion: unfortunately this issue seems to be caused by a bug in libcdio-paranoia (probably similar to the one reported in libcdio/libcdio-paranoia#14).

@JoeLametta JoeLametta added Upstream Bug The issue is a result of an upstream bug Accepted Accepted issue on our roadmap Priority: medium Medium priority Needed: patch A pull request is required Needed: discussion More discussion needed before anything can be done (or still no agreement has been reached) and removed *paranoia labels Nov 11, 2018
@JoeLametta JoeLametta added this to the 2.0 milestone Nov 13, 2018
@JoeLametta JoeLametta added the Stretch Optional goal for the current sprint (may not be delivered) label Dec 15, 2018
@JoeLametta JoeLametta modified the milestones: 2.0, 1.0 Dec 15, 2018
@Bujiraso
Copy link

Bujiraso commented Mar 28, 2021

I'm also seeing this one and left some related comments on #474

On Antichrist Superstar, with 99 tracks...

$ cd-paranoia --stderr-progress --sample-offset=6 --force-cdrom-device /dev/sr0 -B 99-99 $HOME/track_99.wav
Sending all callback output to stderr for wrapper script
cdparanoia III release 10.2 libcdio 2.1.0 x86_64-pc-linux-gnu
(C) 2001 Monty <[email protected]> and Xiphophorus
(C) 2004, 2005, 2008 Rocky Bernstein <[email protected]>
(C) 2014 Robert Kausch <[email protected]>

Report bugs to [email protected]

Selected span contains non audio track at track 101.  Aborting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Accepted issue on our roadmap Bug Generic bug: can be used together with more specific labels Needed: discussion More discussion needed before anything can be done (or still no agreement has been reached) Needed: patch A pull request is required Priority: medium Medium priority Stretch Optional goal for the current sprint (may not be delivered) Upstream Bug The issue is a result of an upstream bug
Projects
None yet
Development

No branches or pull requests

3 participants