After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 510322 - x-content/* support
x-content/* support
Status: RESOLVED FIXED
Product: sound-juicer
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Sound Juicer Maintainers
Sound Juicer Maintainers
Depends on:
Blocks: 510319
 
 
Reported: 2008-01-18 04:39 UTC by David Zeuthen (not reading bugmail)
Modified: 2008-01-18 16:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (545 bytes, patch)
2008-01-18 05:54 UTC, Matthias Clasen
committed Details | Review

Description David Zeuthen (not reading bugmail) 2008-01-18 04:39:44 UTC
Please include support for the new x-content/* machinery available in Nautilus.
Specifically, adding support for the types

 x-content/audio-cdda
 x-content/audio-dvd

would make sense.

Please see bug 510319 for details about x-content/*. Thanks.
Comment 1 Matthias Clasen 2008-01-18 05:54:21 UTC
Created attachment 103117 [details] [review]
patch
Comment 2 David Zeuthen (not reading bugmail) 2008-01-18 06:06:43 UTC
Btw, there's a slight conflict here. Currently running sound-juicer, at least on Fedora, with the gvfs cdda backend active is not working. This is because s-j uses libcdparanoia which (errornously) uses O_EXCL (on Linux) to lock the drive. The way to fix this is to avoid using O_EXCL (or use cdda:/// directly to read the PCM data). One way to fix this is to use libcdio_paranoia from libcdio

 http://www.gnu.org/software/libcdio/

and in addition this will make things work on mixed discs. There's some more detail on this topic here

http://mail.gnome.org/archives/gtk-devel-list/2007-December/msg00228.html

and surrounding messages.

Btw, in addition, using libcdio, would gain you support for other OS's than Linux cf. this message

http://mail.gnome.org/archives/gnome-vfs-list/2006-May/msg00030.html
Comment 3 Ross Burton 2008-01-18 08:30:05 UTC
There is a plan to switch to cdio, I let it drop this release cycle but maybe I should switch today and see how it works.
Comment 4 Ross Burton 2008-01-18 09:50:35 UTC
Right, sound-juicer does this:

  priv->cdsrc = gst_element_make_from_uri (GST_URI_SRC, "cdda://1", "cd_src");

So it's up to GStreamer to decide what element to return.  Most people have two choices: cdiocddasrc or cdparanoiasrc.  cdparanoiasrc has a higher rank than cdiocddasrc, so that gets chosen.  The cdparanoia element is also a -base plugin, cdio is -good.

I'd like to see the cdio plugin promoted over the cdparanoia plugin for reasons of saner code and better cross platform support, but this is a gstreamer issue.

That said, if Fedora wanted to manually boost the rank of the cdio plugin to see what happened, I'd love to know. :)
Comment 5 Ross Burton 2008-01-18 10:03:03 UTC
I thought I'd just hack this and removed the cdparanoia plugin briefly.  libcdio is *vastly* slower, instead of ripping at 14x I'm only getting 4x.  GStreamer doesn't let me control the level of paranoia, so maybe its always a full paranoia.
Comment 6 Ross Burton 2008-01-18 10:39:53 UTC
Oh, and back to the real point of this bug: I'm not sure that adding DVD would work, as far as I know cdparanoia and cdio can't rip DVD-Audio disks.
Comment 7 Ross Burton 2008-01-18 10:48:09 UTC
Committed the CD audio content type, I've left of DVD-audio until someone can demonstrate that they work.  I'm pretty certain they won't.
Comment 8 David Zeuthen (not reading bugmail) 2008-01-18 16:57:38 UTC
Thanks for committing this. I agree about x-content/audio-dvd, that was more wishful thinking on my part ;-)