Update audio/cdparanoia from cdparanoia-III-alpha9.8 to cdparanoia-III-10.2 . https://xiph.org/paranoia/ http://downloads.xiph.org/releases/cdparanoia/cdparanoia-III-10.2.src.tgz https://svn.xiph.org/trunk/cdparanoia/
Yes, I'm having a half-baked (actually more than half: it is runnable) sitting im my tree, but had experiences weird segmentation faults that did not occur with current (previous) version. I know it's been a while, I really need to get my act together and finish it. Thanks for bugging!
Maybe post the link to your tree in here or put the code somewhere (github?) and post to multimedia@ maybe some other people will look at it with you. I'm sure you will find these faults :) As far as port version things... Some other OS are based on cdparanoia RELEASE, but are carrying some patches that have since been committed to cdparanoia HEAD, which is redundant. Since cdparanoia is small and inactive, there would be no need to do that, or to maintain a -dev version of it in ports. So I would actually base the port on cdparanoia HEAD in the following way and order: the RELEASE tarball above + one patch file to HEAD + patch files needed to port the HEAD to FreeBSD + any remaining bug and feature patches from other OS that are not in HEAD. cdparanoia HEAD r17289 2010-06-11 cdparanoia RELEASE 10.2 r15302 2008-09-11 svn log https://svn.xiph.org/trunk/cdparanoia/ svn diff -r 15302:HEAD https://svn.xiph.org/trunk/cdparanoia/ > HEAD.patch Here are the other major independant OS repos for reference, review, and inclusion. Note the kFreeBSD patch, and that both NetBSD and all the Linux are on 10.2 (the rest of BSD are on 9.8). http://cvsweb.netbsd.org/bsdweb.cgi/pkgsrc/audio/cdparanoia/ https://svnweb.freebsd.org/ports/head/audio/cdparanoia/ https://github.com/DragonFlyBSD/DPorts/tree/master/audio/cdparanoia http://cvsweb.openbsd.org/cgi-bin/cvsweb/ports/audio/cdparanoia/ https://anonscm.debian.org/cgit/pkg-opt-media/cdparanoia.git https://anonscm.debian.org/cgit/pkg-opt-media/cdparanoia.git/tree/debian/patches/05-kfreebsd.patch https://projects.archlinux.org/svntogit/packages.git/?h=packages/cdparanoia https://pkgs.fedoraproject.org/cgit/cdparanoia.git/ https://build.opensuse.org/package/show/openSUSE:Factory/cdparanoia https://svnweb.mageia.org/packages/cauldron/cdparanoia/current/
Any advance here?
ping!
I recently decided to give this update a try on my own, independent of the maintainers work. I have an experimental version of cdparanoia-III-10.2 that seems to work fine on my machine. I am still testing with some audio CDs that triggered reproducible SCSI errors using 9.8 (this was the reason I decided to work on this issue) and it looks good so far. In other words: I don't feel ready to officially submit this update right now (in the end this stuff is surprisingly low level), but if someone wants to try the update and provide feedback or if danfe is interested in merging my stuff into their work, feel free: https://github.com/blabber/freebsd-ports-wip/tree/master/audio/cdparanoia
Created attachment 206100 [details] patch to update audio/cdparanoia to the latest released version
audio/cdparanoia: Update to cdparanoia-III-10.2 (3.10.2) The attached patch updates cdparanoia to the latest release. Additionally the patch adds support for audio CDs containing data tracks. Three patches committed to the upstream SVN are also included: -r15314: Optimizes autosense and fixes a bug in verify_read_command. -r15337: fixes the "-p" option. -r15338: fixes build on gcc architectures. * portlint: looks fine * poudriere: amd64 12.0 and amd64 11.3 are ok I ripped several enhanced CDs (audio + data) using this version without any issues. I also managed to rip a CD that reproducible triggered a SCSI transport error with the old version. Another CD that reported a scratch using the old version, reported only jitter using this version.
Thank you for the QA'd update Tobias!
A final report from my side: I ripped 147 audio CDs and enhanced audio cds using the attached patch. I am quite confident, that the update works fine. My main issue is, that I only have a single drive to test, but I got a success report by one person on IRC who volunteered to test the update.
(In reply to tobias.rehbein from comment #9) > I ripped 147 audio CDs and enhanced audio CDs using the attached patch. > I am quite confident, that the update works fine. I did a ripping of my CD collection back in the time when actively testing 10.2 update, and discovered that several CDs did not rip identically compared to EAC, albeit EAC log reported 100% match with AccurateRip database on a calibrated drive (same hardware, just booted under Windoze). Hex-diff showed some bytes/words were swapped here and there, as well as some other minor artefacts AFAIR. Did you see anything alike? Existing version did not exhibit these issues, thus blocking the update. I need to retest it again, since time had passed and things might have changed.
(In reply to Alexey Dokuchaev from comment #10) Oh, sorry. I don't have a windows system and therefore can't compare. What I have checked is, that the old and the new version produce identical WAV files if cdparanoia signals a perfect rip. But I have done the comparison just for a few CDs.
Any updates on this one?
Can we close this by now since there's no upstream development and I guess bugs remain? A wild guess is that it may have something to do with the new drive cache changes/workarounds? @danfe If you can, can you give perhaps give audio/cyanrip a quick try to see if it exhibits the same issue? It works fine for me at least
I am still using this patch together with audio/abcde without any issues. That said it is okay for me too close this issue, if there are doubts. I will just keep this in my local ports tree (as I did for six years now).
(In reply to Tobias Rehbein from comment #14) s/six years/three years/
(In reply to Daniel Engberg from comment #13) Thanks for the pointer, I'll check but I'd rather keep this PR open, as it's still valid, just low priority ATM. I still intend to ensure there would be no regressions and update the port one day.
Created attachment 247627 [details] patch to update audio/cdparanoia to the latest released version Updated the patch to one created using `git format-patch`. I also removed some new issues created by stage-qa and portlint: * strip binary * move WWW from pkg-descr to Makefile
Hi Tobias, I'd like to test this one but there seems to be a couple of stray files? extra-patch-upstream-patch-r15314 extra-patch-upstream-patch-r15337 extra-patch-upstream-patch-r15338 patch-main.c patch-paranoia_overlap.c Thanks for working on this! Best regards, Daniel
Created attachment 247628 [details] patch to update audio/cdparanoia to the latest released version Hi Daniel. Interesting, I don't know what happened, but I rerolled the patch.
Thanks! Appears to work on my end and results seems to similar to what cyanrip produces but offset seems a bit different which is to be expected I guess? Best regards, Daniel
(In reply to Daniel Engberg from comment #20) To be honest, I don't know. I submitted my initial patch 2019, when I was ripping my whole CD collection and the version in ports failed on some CDs. I ripped a total of 147 CDs and was happy with the result. That's all I can say ;) I just happened to look through the PRs I am involved in and got the idea to update the patch, to make life easier for others.