Bug 226065

Summary: rhythmbox/totem-plparser doesn't recognize Audio CDs at all
Product: [openSUSE] openSUSE 10.2 Reporter: Andreas Hanke <andreas.hanke>
Component: GNOMEAssignee: Joe Hargadon <jhargadon>
Status: RESOLVED FIXED QA Contact: E-mail List <qa-bugs>
Severity: Major    
Priority: P5 - None    
Version: RC 5   
Target Milestone: ---   
Hardware: x86   
OS: Linux   
Whiteboard:
Found By: Other Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---
Attachments: Extracted from upstream CVS
non-hal case
hal case

Description Andreas Hanke 2006-12-05 14:30:08 UTC 
rhythmbox is supposed to show an Audio CD in the left "Source" pane, next to the "Play Queue"/"Library"/"Radio"/"Podcasts" items.

But this does never happen, the Audio CD does not show up, and is not accessible via "Music" -> "Import <whatever>" either.

This *might* be related to bug 225968, but I'm not sure. CD playback with Sound-Juicer or a simple gstreamer pipeline works, so it seems that the CD is simply not found by rhythmbox.
Comment 1 Andreas Hanke 2006-12-05 16:02:51 UTC 
This is clearly totem's fault.

I have compiled totem myself from CVS HEAD and set the LD_LIBRARY_PATH so that rhythmbox picks libtotem-plparser.so.1 from the new CVS totem build.

Result: The Audio CD shows up without any problems in rhythmbox and is played properly.
Comment 2 Andreas Hanke 2006-12-05 16:44:29 UTC 
Created attachment 108379 [details]
Extracted from upstream CVS

I could track it down to this upstream change; will try to reduce it even more now.
Comment 3 Andreas Hanke 2006-12-05 17:29:26 UTC 
I *think* I found it, but in a totally different place:

Your totem package is built without hal support because of http://bugzilla.gnome.org/show_bug.cgi?id=365387 - not good
Comment 4 Andreas Hanke 2006-12-06 00:15:48 UTC 
Yes, that's it. Testing shows that it works without any problems if hal is enabled.

The attachment above is far too much because it makes changes to the hal case which are not necessary because enabling hal is already enough.

For future releases, you should simply upgrade totem to >= 2.17.2, where this very unfortunate typo, which causes hal to be disabled, is fixed.

For openSUSE 10.2, I don't know if you're going to do anything, but there would be two solutions: Enable hal or fix the non-hal case. Enabling hal should be relatively safe because it doesn't add extra dependencies: totem does already (indirectly) depend on hal (via gnome-vfs2), the problem is that it doesn't use it.
Comment 5 Andreas Hanke 2006-12-06 00:57:13 UTC 
Created attachment 108442 [details]
non-hal case
Comment 6 Andreas Hanke 2006-12-06 00:57:32 UTC 
Created attachment 108443 [details]
hal case
Comment 7 JP Rosevear 2006-12-11 17:32:50 UTC 
We should definitely enable hal and ship an update.
Comment 8 Joe Hargadon 2006-12-13 21:01:16 UTC 
Gary, please create a swampid for this.
Comment 9 Gary Ekker 2006-12-27 23:50:43 UTC 
Joe, you need AJ to create a swamp for 10.2.
Comment 10 Joe Hargadon 2006-12-28 22:38:32 UTC 
AJ, please create a swampid for this.
Comment 12 Andreas Jaeger 2007-01-12 15:58:54 UTC 
swampid is: MaintenanceTracker-7878
Comment 13 Joe Hargadon 2007-01-12 17:28:57 UTC 
I have submitted this package and the patchinfo file.
Comment 14 Anja Stock 2007-01-16 11:44:17 UTC 
released