Audio preview of CD tracks fails if totem is not installed

Bug #607793 reported by Olivier Tilloy
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Nautilus
Fix Released
Medium
nautilus (Debian)
Fix Released
Unknown
nautilus (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: nautilus

To perform audio preview, Nautilus uses totem-audio-preview. If not present (i.e. if totem is not installed), it falls back on gst-launch-0.10 with a playbin pipeline. Unfortunately playbin doesn't understand the cdda:// URIs when trying to preview tracks on an audio CD.

Steps to reproduce:
1) Uninstall totem:
    sudo apt-get remove totem
2) Install gstreamer0.10-tools:
    sudo apt-get install gstreamer0.10-tools
3) Make sure audio previews are active:
    gconftool-2 --type string --set /apps/nautilus/preferences/preview_sound "always"
4) Insert an audio CD, when prompted what to do choose "Open Folder"
5) Hover the mouse cursor over one of the tracks in nautilus (typically "Track n.wav")

Expected result: after a delay (about 1 second), the track starts playing.
Current result: nothing happens.

Note that when totem is installed, the preview works.

ProblemType: Bug
DistroRelease: Ubuntu 10.04
Package: nautilus 1:2.30.1-0ubuntu1.1
ProcVersionSignature: Ubuntu 2.6.32-23.37-generic 2.6.32.15+drm33.5
Uname: Linux 2.6.32-23-generic i686
NonfreeKernelModules: nvidia wl
Architecture: i386
Date: Tue Jul 20 16:14:38 2010
EcryptfsInUse: Yes
ProcEnviron:
 LANG=fr_FR.utf8
 SHELL=/bin/bash
SourcePackage: nautilus

Related branches

Revision history for this message
Olivier Tilloy (osomon) wrote :
Revision history for this message
Olivier Tilloy (osomon) wrote :

The issue is with the way nautilus builds the URI to pass to playbin. The attached patch mimics totem's behaviour to build URIs that playbin is happy with.

tags: added: patch
Revision history for this message
Pedro Villavicencio (pedro) wrote :

Thanks for the report.

Changed in nautilus (Ubuntu):
status: New → Triaged
importance: Undecided → Low
Changed in nautilus:
status: Unknown → New
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for your work there, do you need that uploaded in maverick now or are you fine waiting the next upload?

Revision history for this message
Olivier Tilloy (osomon) wrote :

I'm perfectly fine waiting for the next upload, there's no hurry considering the importance of the bug.

Changed in nautilus (Debian):
status: Unknown → New
Changed in nautilus (Debian):
status: New → Confirmed
Revision history for this message
Sebastien Bacher (seb128) wrote :

Upstream has comments on the change, could you update the patch and subscribe the sponsors again once it's done?

Revision history for this message
Olivier Tilloy (osomon) wrote :

Sure, updating the patch is on my list.

Revision history for this message
Olivier Tilloy (osomon) wrote :

Updated patch submitted upstream.

tags: added: patch-forwarded-upstream
removed: lucid patch
Revision history for this message
Benjamin Drung (bdrung) wrote :

unsubscribing ubuntu-sponsors. Sebastian, the ubuntu-sponsor team is for sponsoring debdiffs (in this case after getting it fixed upstream first). subscribed ubuntu-reviewers instead.

Revision history for this message
Sebastien Bacher (seb128) wrote :

Benjamin, it's perfectly acceptable to sponsor a patch rather than a debdiff

Revision history for this message
Sebastien Bacher (seb128) wrote :

unsubscribing the review team again, Olivier can you just subscribe sponsors back when upstream is happy with the change?

Revision history for this message
Olivier Tilloy (osomon) wrote :

Sure.

Changed in nautilus:
importance: Unknown → Medium
Revision history for this message
Olivier Tilloy (osomon) wrote :
Revision history for this message
Sebastien Bacher (seb128) wrote :

thank you for your work on that, it will be in the next version update

Changed in nautilus (Ubuntu):
status: Triaged → Fix Committed
Changed in nautilus:
status: New → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package nautilus - 1:2.32.0-0ubuntu4

---------------
nautilus (1:2.32.0-0ubuntu4) natty; urgency=low

  * debian/patches/90_git_audio_preview_url.patch: fix audio preview urls, so
    the feature work without totem, thanks Olivier Tilloy (lp: #607793)
 -- Sebastien Bacher <email address hidden> Fri, 05 Nov 2010 12:15:20 +0100

Changed in nautilus (Ubuntu):
status: Fix Committed → Fix Released
Changed in nautilus (Debian):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.