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 653493 - Add CDG demuxing/decoding support
Add CDG demuxing/decoding support
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-libav
git master
Other Linux
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2011-06-27 14:43 UTC by John Carr
Modified: 2018-11-03 12:55 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add CDG support (3.17 KB, patch)
2011-10-10 21:30 UTC, Guillaume Desmottes
needs-work Details | Review
cdg.c: set AV_PKT_FLAG_KEY flag (714 bytes, patch)
2011-10-10 21:31 UTC, Guillaume Desmottes
rejected Details | Review

Description John Carr 2011-06-27 14:43:14 UTC
Using ggst-ffmpeg 0.10.11 and gst 0.10.32.

I am using a sample from:

http://www.powerkaraoke.com/download/NobodyKnowsYou.cdg

Totem is unable to open it.

I attempted to write a gst-launch pipeline and that yielded an Internal Data Error.

I used:

gst-launch-0.10 --gst-debug ffmpeg:5 filesrc location=/tmp/NobodyKnowsYou.cdg ! ffdemux_cdg ! ffdec_cdgraphics ! ximagesink

The error seems to be coming from gst_ffmpegdata_peek which claims it is pulling 21600 bytes then fails with "Got -3 (unexpected) return result 0".
Comment 1 David Schleef 2011-06-30 05:29:43 UTC
The CDG demuxer was recently disabled because it has never worked.  Marking as enhancement.
Comment 2 Guillaume Desmottes 2011-08-12 17:01:19 UTC
(In reply to comment #1)
> The CDG demuxer was recently disabled because it has never worked.  Marking as
> enhancement.

I'd be interested trying to work on this. Could you please point me the commit disabling it? I didn't manage to find it.
Comment 3 Guillaume Desmottes 2011-08-12 18:01:05 UTC
Looking at ./gst-libs/ext/libav/libavformat/cdg.c ffmpeg doesn't seem to do much demuxing; am I right?

Would be better/easier to write a gst element implementing a cdg demuxer from scratch rather than using ffmpeg?
Comment 4 Guillaume Desmottes 2011-08-12 22:44:23 UTC
(In reply to comment #3)
> Looking at ./gst-libs/ext/libav/libavformat/cdg.c ffmpeg doesn't seem to do
> much demuxing; am I right?

Hum actually I think it does. So what we need to is to get the decoder working with gst.
Comment 5 Guillaume Desmottes 2011-10-10 21:30:08 UTC
Thanks to Olivier I finally managed to get this working! I had to patch ffmpeg to get it working but I'm not sure if that's the proper fix.
Comment 6 Guillaume Desmottes 2011-10-10 21:30:37 UTC
Created attachment 198738 [details] [review]
Add CDG support
Comment 7 Guillaume Desmottes 2011-10-10 21:31:16 UTC
Created attachment 198739 [details] [review]
cdg.c: set AV_PKT_FLAG_KEY flag
Comment 8 Edward Hervey 2011-10-11 07:11:53 UTC
Review of attachment 198739 [details] [review]:

Please submit this patch to libav, we'll pick the proper checkout once it's in there.
Comment 9 Edward Hervey 2011-10-11 07:13:10 UTC
Other patch looks ok
Comment 10 Guillaume Desmottes 2011-10-11 13:44:46 UTC
(In reply to comment #8)
> Review of attachment 198739 [details] [review]:
> 
> Please submit this patch to libav, we'll pick the proper checkout once it's in
> there.

Sure, but I wanted to have your advice on it first; it's not like I'm really understanding any of this. :)
Comment 11 Guillaume Desmottes 2011-10-11 23:06:31 UTC
(In reply to comment #8)
> Review of attachment 198739 [details] [review]:
> 
> Please submit this patch to libav, we'll pick the proper checkout once it's in
> there.

I opened https://ffmpeg.org/trac/ffmpeg/ticket/543
Comment 12 Guillaume Desmottes 2011-10-11 23:31:19 UTC
Also reported to http://bugzilla.libav.org/show_bug.cgi?id=55
Comment 13 Sebastian Dröge (slomo) 2013-07-23 13:09:54 UTC
Any news here?
Comment 14 Guillaume Desmottes 2013-07-24 08:19:33 UTC
I'm not planning to work further on this in the near future so feel free to take it over.
Comment 15 Sebastian Dröge (slomo) 2013-07-24 09:48:46 UTC
Not planning to take it over, just wondering if there's still interest or if we should just close this
Comment 16 Guillaume Desmottes 2013-07-24 10:41:36 UTC
Well, I still think CDG support in GStreamer would be good.
Comment 17 Sebastian Dröge (slomo) 2014-01-06 09:20:59 UTC
Sure, would be nice to have. But keeping a feature request open for something nobody is interested to implement is not very useful :)
Comment 18 GStreamer system administrator 2018-11-03 12:55:36 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gst-libav/issues/6.