.
Created attachment 340506 [details] [GStreamer] Update to GStreamer 1.14.0 in jhbuild And update the patches, removing the ones that have been merged upstream.
Comment on attachment 340506 [details] [GStreamer] Update to GStreamer 1.14.0 in jhbuild View in context: https://bugs.webkit.org/attachment.cgi?id=340506&action=review > Tools/ChangeLog:14 > + * gstreamer/patches/gst-plugins-good-0006-qtdemux-add-context-for-a-preferred-protection.patch: Removed. > + * gstreamer/patches/gst-plugins-good-0008-qtdemux-also-push-buffers-without-encryption-info-in.patch: Removed. Those 2 are still in Bugzilla hibernation. I locally rebased them against git master, I can share them if you want.
(In reply to Philippe Normand from comment #2) > Comment on attachment 340506 [details] > [GStreamer] Update to GStreamer 1.14.0 in jhbuild > > View in context: > https://bugs.webkit.org/attachment.cgi?id=340506&action=review > > > Tools/ChangeLog:14 > > + * gstreamer/patches/gst-plugins-good-0006-qtdemux-add-context-for-a-preferred-protection.patch: Removed. > > + * gstreamer/patches/gst-plugins-good-0008-qtdemux-also-push-buffers-without-encryption-info-in.patch: Removed. > > Those 2 are still in Bugzilla hibernation. I locally rebased them against > git master, I can share them if you want. Erg, forgot to re add updated version, sorry.
Created attachment 340510 [details] Fix mixup with patches.
Comment on attachment 340510 [details] Fix mixup with patches. View in context: https://bugs.webkit.org/attachment.cgi?id=340510&action=review > Tools/gstreamer/jhbuild.modules:78 > + <patch file="gst-plugins-good-0001-souphttpsrc-cookie-jar-and-context-query-support.patch" strip="1"/> > + <patch file="gst-plugins-good-0002-qtdemux-add-context-for-a-preferred-protection.patch" strip="1"/> > + <patch file="gst-plugins-good-0003-qtdemux-also-push-buffers-without-encryption-info-in.patch" strip="1"/> Please, add a comment before any patch explaining why they are needed, where they came from and whether they will be included in a future gst release. I know they were here before, but now that we are updating gst, we should document the patches we are using.
(In reply to Carlos Garcia Campos from comment #5) > Please, add a comment before any patch explaining why they are needed, where > they came from and whether they will be included in a future gst release. I > know they were here before, but now that we are updating gst, we should > document the patches we are using. I think we need to actually actively remove any patches that are either not upstream, or not on a patch to being upstreamed in the near future. Keeping patches here over a major version upgrade is *not* OK, it indicates not enough effort to upstream the patches between 1.12 and 1.14.
(In reply to Michael Catanzaro from comment #6) > (In reply to Carlos Garcia Campos from comment #5) > > Please, add a comment before any patch explaining why they are needed, where > > they came from and whether they will be included in a future gst release. I > > know they were here before, but now that we are updating gst, we should > > document the patches we are using. > > I think we need to actually actively remove any patches that are either not > upstream, or not on a patch to being upstreamed in the near future. Keeping > patches here over a major version upgrade is *not* OK, it indicates not > enough effort to upstream the patches between 1.12 and 1.14. Well those are still being worked on, I will document it more.
1.14.1 is out today btw, seize the opportunity? :)
(In reply to Philippe Normand from comment #8) > 1.14.1 is out today btw, seize the opportunity? :) ++
Created attachment 340962 [details] - Remove now unnecessary `gstreamer/patches/gst-plugins-good-0001-souphttpsrc-cookie-jar-and-context-query-support.patch` - All other patches have been merged now, specify in the ChangeLog
Created attachment 340963 [details] We are now using GStreamer 1.14.1 - specify in the ChangeLog
I'm afraid we still need patches that are not part of 1.14.1 :/
(In reply to Philippe Normand from comment #12) > I'm afraid we still need patches that are not part of 1.14.1 :/ I am keeping the 2 patches that are still required, namely: > gstreamer/patches/gst-plugins-good-0002-qtdemux-add-context-for-a-preferred-protection.patch: Renamed from Tools/gstreamer/patches/gst-plugins-good-0006-qtdemux-add-context-for-a-preferred-protection.patch > This patch has now been merged in GStreamer master and will be there in GStreamer 1.16. > gstreamer/patches/gst-plugins-good-0003-qtdemux-also-push-buffers-without-encryption-info-in.patch: Renamed from Tools/gstreamer/patches/gst-plugins-good-0008-qtdemux-also-push-buffers-without-encryption-info-in.patch. > This patch has now been merged in GStreamer master and will be there in GStreamer 1.16.
Created attachment 340968 [details] Add upstream commit ID in the .jhbuild file
Comment on attachment 340968 [details] Add upstream commit ID in the .jhbuild file Clearing flags on attachment: 340968 Committed r232060: <https://trac.webkit.org/changeset/232060>
All reviewed patches have been landed. Closing bug.
<rdar://problem/40447628>
This patch produces regressions in the LayoutTests. All the tests: imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-* crash. https://build.webkit.org/builders/WPE%20Linux%2064-bit%20Release%20%28Tests%29/builds/7887/steps/layout-test/logs/stdio
(In reply to Yacine Bandou from comment #18) > This patch produces regressions in the LayoutTests. > > All the tests: > imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-* > crash. > > https://build.webkit.org/builders/WPE%20Linux%2064- > bit%20Release%20%28Tests%29/builds/7887/steps/layout-test/logs/stdio I guess this to be investigated.