libcdio-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Libcdio-devel] Any netbsd + libcdio people out there?


From: Rocky Bernstein
Subject: [Libcdio-devel] Any netbsd + libcdio people out there?
Date: Sun, 7 Oct 2018 10:28:30 +0000

Edd Barrett has been doing great work to get libcdio working on OpenBSD.

He came across the following problem which he has fixed in OpenBSD. Is this
also relevant to NetBSD?

>From Edd:

> I took my branch [
> https://github.com/vext01/libcdio/commits/openbsd_fixes_to_master] and
> built audacious and mpv -- two music players with
> libcdio support -- and tried playing CDs. They behaved really oddly.
> The CDs that did play didn't start at the beginning, and many didn't
> work at all.
>

 and then later:

Just to follow up, after some hours of debugging, I've found the
> culprit.
>
> diff --git a/lib/driver/netbsd.c b/lib/driver/netbsd.c
> index ed3f00a9..7c6fe636 100644
> --- a/lib/driver/netbsd.c
> +++ b/lib/driver/netbsd.c
> @@ -331,7 +331,8 @@ _cdio_read_toc(_img_private_t *_obj)
>                  return false;
>          }
>
> -        req.address_format = CD_MSF_FORMAT;
> +        //req.address_format = CD_MSF_FORMAT;
> +        req.address_format = CD_LBA_FORMAT;
>          req.starting_track = FIRST_TRACK_NUM;
>          req.data_len = (TOTAL_TRACKS + 1) /* leadout! */
>                  * sizeof(struct cd_toc_entry);k
>

> I've just managed to rip a whole CD using cd-paranoia with that change.
>
> I wonder if it is incorrect for NetBSD too?
>
> I'll guard it with an `#if defined(__OpenBSD__)` for now:
>
> https://github.com/vext01/libcdio/commit/e2f3919357b76d9fadb3d57f20d2b1e8198f32d1
>

Any thoughts from the NetBSD users/community ?


reply via email to

[Prev in Thread] Current Thread [Next in Thread]