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 693628 - mxfdemux: add a second MXF UL for uncompressed audio; ignore missing metadata descriptors
mxfdemux: add a second MXF UL for uncompressed audio; ignore missing metadata...
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: 1.1.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-02-12 02:49 UTC by David Schleef
Modified: 2013-03-15 08:47 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch 1 (734 bytes, patch)
2013-02-12 02:49 UTC, David Schleef 		accepted-commit_now Details | Review
patch 2 (1.36 KB, patch)
2013-02-12 02:49 UTC, David Schleef 		accepted-commit_now Details | Review

Description David Schleef 2013-02-12 02:49:05 UTC 
Created attachment 235760 [details] [review]
patch 1

0001-mxfdemux-Ignore-missing-metadata-descriptors.patch

A file I'm dealing with right now has missing descriptors for metadata.  VLC doesn't seem to mind this, and I don't see a reason to require resolution of metadata.  This seemed to be a simple fix.


0002-mxfdemux-Add-a-second-MXF-UL-for-uncompressed-audio.patch

I can't find any information about this UL, but it appeared in a file I'm dealing with right now.
Comment 1 David Schleef 2013-02-12 02:49:49 UTC 
Created attachment 235761 [details] [review]
patch 2
Comment 2 David Schleef 2013-02-12 02:59:01 UTC 
Found the ID in S382m-2007.pdf on (labeled) page 11.
Comment 3 Sebastian Dröge (slomo) 2013-02-12 09:04:43 UTC 
Looks good to me (both).

If descriptors can't be resolved this should only be fatal if none was resolved or the resolved ones don't contain the informations needed for the mapping.
Comment 4 David Schleef 2013-03-15 07:06:29 UTC 
pushed.  (Including, accidentally, the NOT FOR MASTER part of the commit log.  oops)