Re: [PATCH 2/7] gstreamer1.0-plugins-base: upgrade to version 1.6.0

[Carlos Rafael Giani <dv@pseudoterminal.org>]

Am 2015-09-28 um 03:07 schrieb Khem Raj:
> On Sun, Sep 27, 2015 at 2:48 PM, Carlos Rafael Giani
> <dv@pseudoterminal.org> wrote:
>> A number of patches had to be deleted in order for the recipe to work.
>>
>> Patches which cannot be applied cleanly to 1.6.0 and are nontrivial to fix:
>> * 0001-basetextoverlay-make-memory-copy-when-video-buffer-s.patch
>> * do-not-change-eos-event-to-gap-event2.patch
>> * do-not-change-eos-event-to-gap-event3.patch
>> * do-not-change-eos-event-to-gap-event-if.patch
>> * handle-audio-video-decoder-error.patch
>> * taglist-not-send-to-down-stream-if-all-the-frame-cor.patch
>>
> has these been dropped ? if so then what do we lose.

We lose nothing that hasn't either been included already or needs to be 
reworked anyway.

There are two things I overlooked though:

* The do-not-change-eos-event-to-gap-event* patches aren't needed 
anymore, because the authors have sent in a completely different patch, 
which has been included in upstream.
* The 0002-gstplaysink-don-t-set-async-of-custom-text-sink-to-f.patch , 
which hasn't been deleted so far, has been rejected by upstream. I 
noticed there is a "denied" upstream-status available, but I would 
rather delete this patch as well, especially since the discussion in 
upstream indicates that this is more i.MX specific. Opinions?

The other patches in the list above, however, need to be reworked to 
apply against 1.6.
0001-basetextoverlay-make-memory-copy-when-video-buffer-s.patch has been 
rejected by upstream,
handle-audio-video-decoder-error.patch is under discussion.
taglist-not-send-to-down-stream-if-all-the-frame-cor.patch hasn't been 
looked at yet by upstream.

Since these no longer apply cleanly against 1.6.0, we anyway have no 
option but to delete them and wait until the authors reworked them.