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

[Zhu Carol]

Hi Otavio,
   The 1.4.2 fix is ugly, so [OE-core] [poky][PATCH v3 5/6] gstreamer1.0-plugins-base: Add videofilter related patch is used to revert it.
   Actually these patches are all back port from 1.5.1, I will change the Upstream-Status and merge these two patches together.

B.R.
Carol

-----Original Message-----
From: Otavio Salvador [mailto:otavio.salvador at ossystems.com.br] 
Sent: Tuesday, July 28, 2015 7:19 PM
To: Zhu Yuqing-B54851
Cc: Patches and discussions about the oe-core layer
Subject: Re: [OE-core] [poky][PATCH v2 4/6] gstreamer1.0-plugins-base: Add video-frame related patch

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