Bug 185731 - [MSE][GStreamer] Add patch to jhbuild: matroskademux: Allow Matroska headers to be read more than once
Summary: [MSE][GStreamer] Add patch to jhbuild: matroskademux: Allow Matroska headers ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Alicia Boya García
URL:
Keywords:
: 179241 (view as bug list)
Depends on:
Blocks:
 
Reported: 2018-05-17 08:40 PDT by Alicia Boya García
Modified: 2018-10-01 07:39 PDT (History)
6 users (show)

See Also:


Attachments
Patch (4.06 KB, patch)
2018-05-17 08:42 PDT, Alicia Boya García
no flags Details | Formatted Diff | Diff
Patch (21.84 KB, patch)
2018-09-24 04:50 PDT, Alicia Boya García
calvaris: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alicia Boya García 2018-05-17 08:40:44 PDT
Upstream bug in GStreamer: https://bugzilla.gnome.org/show_bug.cgi?id=793333

This fixes YTTV 35. AppendMultipleInitOpusAudio (and any other use
case where two WebM initialization segments are appended on a row).
Comment 1 Alicia Boya García 2018-05-17 08:42:39 PDT
Created attachment 340589 [details]
Patch
Comment 2 Michael Catanzaro 2018-05-17 09:24:45 PDT
Comment on attachment 340589 [details]
Patch

I don't see enough activity on the upstream Bugzilla (nothing's happened since February) to justify pulling in this patch now. We need to be really careful about pulling in downstream patches that haven't landed upstream yet. Let's get the patch landed upstream first.
Comment 3 Xabier Rodríguez Calvar 2018-05-21 05:10:59 PDT
(In reply to Michael Catanzaro from comment #2)
> I don't see enough activity on the upstream Bugzilla (nothing's happened
> since February) to justify pulling in this patch now. We need to be really
> careful about pulling in downstream patches that haven't landed upstream
> yet. Let's get the patch landed upstream first.

This is not the first patch that is needed for WebKit that is taking long to land. GStreamer community is very busy. Besides this, MSE is not activated by default in GTK+ so harm is done there.

Anyway, we'll try to get this patch landed in GStreamer this week and if that does not happen, I'll r+ this myself because it is needed even when GStreamer upstream does not support it.
Comment 4 Michael Catanzaro 2018-05-21 08:08:38 PDT
We discussed this last week on our internal chat and decided that Philippe and Thibault would review any pending patches and ensure they are committed upstream, and then Alicia can add them to our JHBuild moduleset once that's been handled.
Comment 5 Xabier Rodríguez Calvar 2018-05-22 00:05:19 PDT
(In reply to Michael Catanzaro from comment #4)
> We discussed this last week on our internal chat and decided that Philippe
> and Thibault would review any pending patches and ensure they are committed
> upstream, and then Alicia can add them to our JHBuild moduleset once that's
> been handled.

IMHO we shouldn't remove them in any case. Those patches are needed and will be handled sooner or later (I know it is later than sooner but you know we do our best, no need to put more pressure ;) )
Comment 6 Michael Catanzaro 2018-05-22 07:28:02 PDT
(In reply to Xabier Rodríguez Calvar from comment #5)
> IMHO we shouldn't remove them in any case. Those patches are needed and will
> be handled sooner or later (I know it is later than sooner but you know we
> do our best, no need to put more pressure ;) )

Better late than never: all our current patches are now upstream, so our GStreamer patch situation is now in a perfect state, as far as I'm concerned!

Alicia, feel free to add any other patches you need once they've first landed upstream, and complain very loudly if you're having trouble getting reviews for that. The ultimate goal is for your work to land, after all. I think we'll all be happier about the situation once that happens.
Comment 7 Michael Catanzaro 2018-05-22 07:29:01 PDT
(In reply to Michael Catanzaro from comment #6)
> Better late than never: all our current patches are now upstream, so our
> GStreamer patch situation is now in a perfect state, as far as I'm concerned!

(Based on Thibault's comment https://bugs.webkit.org/show_bug.cgi?id=185689#c13).
Comment 8 Alicia Boya García 2018-09-24 04:50:11 PDT
Created attachment 350631 [details]
Patch
Comment 9 Alicia Boya García 2018-09-24 05:05:40 PDT
This is finally upstream, I hope there are no objections anymore for adding it to jhbuild.
Comment 10 Xabier Rodríguez Calvar 2018-09-24 06:05:06 PDT
Please, there was something weird when landing this patch because this bug was not closed automatically.
Comment 11 Alicia Boya García 2018-09-24 06:15:08 PDT
Committed r236398: <https://trac.webkit.org/changeset/236398/webkit>
Comment 12 Enrique Ocaña 2018-10-01 07:39:31 PDT
*** Bug 179241 has been marked as a duplicate of this bug. ***