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 587922 - qtmux lacks mapping for Sorenson Video v3
qtmux lacks mapping for Sorenson Video v3
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
0.10.x
Other Linux
: Normal enhancement
: 0.10.18
Assigned To: Thiago Sousa Santos
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-07-06 20:10 UTC by Christian Fredrik Kalager Schaller
Modified: 2009-11-18 20:32 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch for qtdemux (4.90 KB, patch)
2009-10-29 14:22 UTC, Thiago Sousa Santos
none Details | Review
Adds support for SVQ3 in qtmux (9.34 KB, patch)
2009-10-29 14:22 UTC, Thiago Sousa Santos
none Details | Review

Description Christian Fredrik Kalager Schaller 2009-07-06 20:10:45 UTC
Looking into trying to remux a .mov file, I noticed that qtmux is not able to mux
Sorenson Video v3. Caps value is : video/x-svq, svqversion=(int)3

I have a .mov with this video format if wanted.
Comment 1 Thiago Sousa Santos 2009-07-07 15:09:10 UTC
Yes, upload the file somewhere so we can take a look at its fields and map them into qtmux.
Comment 2 Christian Fredrik Kalager Schaller 2009-07-14 12:42:15 UTC
The file is available here:
http://www.linuxrising.org/files/WH_MoC_E3_Intro.mov
Comment 3 Thiago Sousa Santos 2009-10-28 15:34:20 UTC
The problem here with the remuxing is that qtdemux puts the entire stsd atom as codec_data. It should remove all the atom structure and put on the caps only what's 'container' independent info.

I don't know if fixing qtdemux would be an option (ABI stability?), can we do it?

The other option would to hack our way in qtmux by testing if the codec_data is the stsd atom, but this is a hack and it would only work with qtdemux. I wouldn't want to do this unless it is our only resort.
Comment 4 Thiago Sousa Santos 2009-10-29 14:22:09 UTC
Created attachment 146504 [details] [review]
Patch for qtdemux

This patch adds new fields to caps of qtdemux when outputing svq3
Comment 5 Thiago Sousa Santos 2009-10-29 14:22:50 UTC
Created attachment 146505 [details] [review]
Adds support for SVQ3 in qtmux

Adds support for svq3 in qtmux (using the new fields added by the other patch)
Comment 6 Thiago Sousa Santos 2009-11-10 19:56:58 UTC
Notes on this issue:

1) quicktime uses the 'gama' atom, so I say we should get it on qtdemux and put on caps
2) quicktime crashes on some svq3 files that don't have the gama atom, for others, it just works. I have seen no crash so far on files that have the 'gama' atom, for safety, I'd suggest putting the gama atom on all movs with svq3
3) the default value for 'gama' seems to be 0, and not 1.
Comment 7 Thiago Sousa Santos 2009-11-18 20:32:27 UTC
Fixed.

Module: gst-plugins-bad
Branch: master
Commit: edea30e30316c95e0866fb1d566895e25184c781
URL:    http://cgit.freedesktop.org/gstreamer/gst-plugins-bad/commit/?id=edea30e30316c95e0866fb1d566895e25184c781

Author: Thiago Santos <thiago.sousa.santos@collabora.co.uk>
Date:   Thu Oct 29 08:36:02 2009 -0300

qtmux: support for SVQ3



Module: gst-plugins-good
Branch: master
Commit: e35085e5b5191e66efeaa807b877f501d2a26363
URL:    http://cgit.freedesktop.org/gstreamer/gst-plugins-good/commit/?id=e35085e5b5191e66efeaa807b877f501d2a26363

Author: Thiago Santos <thiago.sousa.santos@collabora.co.uk>
Date:   Thu Oct 29 08:29:38 2009 -0300

qtdemux: Add more fields to SVQ3 caps