[OE-core] [poky][PATCH v2 4/6] gstreamer1.0-plugins-base: Add video-frame related patch

[Otavio Salvador]

On Tue, Jul 28, 2015 at 2:23 AM, Zhu Carol <B54851 at freescale.com> wrote:
> Hi Otavio,
>    Really sorry for the Upstream-Status format error, we just kept the same with some patches on fido branch, maybe they are old version. And I still have a little confusion about the third comment. My question are as followed.

No problem; thanks by being responsive with the feedback. Comments below...

...
>> +Upstream Status: Accepted
>> +https://bugzilla.gnome.org/show_bug.cgi?id=736118
>
> The field is:
>
> Upstream-Status: Submitted [https://...]
>
> [Yuqing Zhu-b54851]: the Upstream-Status of this patch is "Accepted", I am not very clear about your comments "Submitted", do you mean I should add the web site in the same line with Upstream-Status?
> Just like this:
> Upstream-Status:  Accepted [https://bugzilla.gnome.org/show_bug.cgi?id=736118]

Accepted would be when upstream says it will apply it for next release
but you don't know when.

Looking at the bug, it seemed it was not been applied yet so it would
be submitted.

> I've read the guide line, here is my understanding:
> Upstream-Status:  Pending
> Upstream-Status:  Submitted [where]
> Upstream-Status:  Accepted
> Upstream-Status:  Backport [from where]
> Upstream-Status:  Denied
> Upstream-Status:  Inappropriate [reason]
>
> Just obey this, right?

Yes.

> So that is
> Upstream-Status:  Accepted
> That's it ?

To be honest, this specific case seems to have been addressed in 1.4.2
in a different way;

http://cgit.freedesktop.org/gstreamer/gst-plugins-base/commit/?h=1.4&id=f711288c7c86c33aed18f52ed6d7787d19b7741d

So this could be dropped.

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750