All of lore.kernel.org
 help / color / mirror / Atom feed
From: Carlos Rafael Giani <dv@pseudoterminal.org>
To: Khem Raj <raj.khem@gmail.com>
Cc: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 2/7] gstreamer1.0-plugins-base: upgrade to version 1.6.0
Date: Mon, 28 Sep 2015 08:53:56 +0200	[thread overview]
Message-ID: <5608E404.9040107@pseudoterminal.org> (raw)
In-Reply-To: <CAMKF1soj1rrFxCG1bQz=F7creT2KBd_SQUOQDXiRUSjq6NA7xA@mail.gmail.com>

Am 2015-09-28 um 08:47 schrieb Khem Raj:
> On Sun, Sep 27, 2015 at 11:44 PM, Carlos Rafael Giani
> <dv@pseudoterminal.org> wrote:
>> 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.
> hmmm, I would have preferred a conclusive situation where we know
> exactly what they fixed and what we dont have when we drop them
> somewhere in commit message, ideally they should be forward ported
> along with recipe upgrades but its ok if you dont feel comfortable and
> want to defer it to original submitters but then please CC the
> submitters so they know whats needed of them

Yep, I already sent them a mail about the necessary rework a month ago, 
when I was first trying out release candidates for 1.6, but I'll give 
them a status update.

As for the patch that got rejected by upstream (but that I didn't 
delete, because it can still be applied), do I delete this one as well? 
Is there a general preference in such situations? I personally would 
prefer to stick with what upstream does.


  reply	other threads:[~2015-09-28  6:53 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-27 21:48 [PATCH 1/7] gstreamer1.0: upgrade to version 1.6.0 Carlos Rafael Giani
2015-09-27 21:48 ` [PATCH 2/7] gstreamer1.0-plugins-base: " Carlos Rafael Giani
2015-09-28  1:07   ` Khem Raj
2015-09-28  6:44     ` Carlos Rafael Giani
2015-09-28  6:47       ` Khem Raj
2015-09-28  6:53         ` Carlos Rafael Giani [this message]
2015-09-28  7:02           ` Khem Raj
2015-09-27 21:48 ` [PATCH 3/7] gstreamer1.0-plugins-good: " Carlos Rafael Giani
2015-10-26 17:25   ` Burton, Ross
2015-09-27 21:48 ` [PATCH 4/7] gstreamer1.0-plugins-bad: " Carlos Rafael Giani
2015-10-26 17:20   ` Burton, Ross
2015-10-27 14:10     ` Carlos Rafael Giani
2015-10-30 18:54       ` Andre McCurdy
2015-11-02 16:56       ` Saul Wold
2015-09-27 21:48 ` [PATCH 5/7] gstreamer1.0-plugins-ugly: " Carlos Rafael Giani
2015-09-27 21:48 ` [PATCH 6/7] gstreamer1.0-libav: " Carlos Rafael Giani
2015-09-27 21:48 ` [PATCH 7/7] gstreamer1.0-rtsp-server: " Carlos Rafael Giani
2015-09-28  9:17 ` [PATCH 1/7] gstreamer1.0: " Burton, Ross

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5608E404.9040107@pseudoterminal.org \
    --to=dv@pseudoterminal.org \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=raj.khem@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.