GNOME Bugzilla – Bug 171600
Audio CD playback doesn't give tracks
Last modified: 2005-04-27 13:59:08 UTC
bacon_video_widget_get_mrls is broken, as it should be giving a list of URIs/MRLs, on for each track on the CD. For example: xine-lib backend: cdda://1 cdda://2 etc. GStreamer backend: cdda://
Note that if the API doesn't fit in what GStreamer can give, we can fix it up (a GList of MRLs would be acceptable).
I can make it fit in, I just didn't know this was how it worked. Lot of legacy around there, I'm affraid.
I tried adding this, and I'm having a problem with track transition. Should it EOS on new-track? Because we currently just continue the next track, back at time=0, but then totem still thinks we're at the previous track (in the playlist). Also, how is metadata done? I don't get song info right now...
Each cdda://X entry is handled like a separate file, and yes, we need EOS to skip to the next track in the playlist. Right now, there's no metadata. I disabled this in the xine-lib backend (xine-lib itself used CDDB), but we do have the Musicbrainz ID for that particular CD, so we'd be able to gather that data.
Created attachment 45405 [details] [review] patch This patch does that, although it allocates an array which is not free'ed, so as said, the BVW API needs a slight change.
That looks good. Could you make the necessary changes to the headers and Totem itself? I'll take care of the xine backend as soon as you're done.
Sure, will do.
Applied, the xine backend was also fixed already.