Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Paul Eggleton <paul.eggleton@linux.intel.com>
To: Otavio Salvador <otavio@ossystems.com.br>
Cc: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: OE-Core release process
Date: Fri, 16 Sep 2011 18:49:52 +0100	[thread overview]
Message-ID: <201109161849.52647.paul.eggleton@linux.intel.com> (raw)
In-Reply-To: <CAP9ODKoVOSsDOKtiv-LExuKbeGciGp4hfC=V5j-zCY8wDNwr3A@mail.gmail.com>

On Friday 16 September 2011 18:18:43 you wrote:
> On Fri, Sep 16, 2011 at 14:16, Paul Eggleton
> 
> <paul.eggleton@linux.intel.com> wrote:
> > On Friday 16 September 2011 18:08:04 Otavio Salvador wrote:
> >> More then once I wanted to pick a patch that was in poky and not yet
> >> merged into oe-core and it wasn't that easy to split it.
> > 
> > Can you give a specific example of this?
> > 
> > Since the split, there have never been any patches intentionally merged
> > into poky's OE-core parts that were not already in OE-core. (There were
> > a few isolated occasions where some accidentally slipped in, these were
> > rectified as soon as they were discovered.)
> 
> They all ended up in oe-core the problem is when they're not yet
> merged but I want to merge them into my tree for use, test or
> anything.

Ah, so what you mean is not that the patches were in poky, but that there 
sometimes were pull requests pointing to a branch that was on top of poky. The 
thing is, the patches are exactly the same.
 
> The combo-layer was one example. It was made available on poky contrib
> tree and I had to manually export the patches instead of pulling it
> into a branch for testing.

Git makes this almost trivially easy though, and in the combo-layer example it 
was only a few patches.

Personally, I always post my OE-core pull requests on top of an OE-core 
branch; doing that means *I* have to manually bring the patches across as files 
and use git-am to apply them onto a branch of OE-core though. I think those of 
us working on Poky know this is the accepted practice and try to do it all of 
the time, occasionally someone might forget or feel that for an RFC patch it's 
not worth the effort; but I think that's the exception rather than the rule.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre



  reply	other threads:[~2011-09-16 17:55 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-16 12:54 OE-Core release process Richard Purdie
2011-09-16 13:29 ` Otavio Salvador
2011-09-16 16:51   ` Richard Purdie
2011-09-16 17:08     ` Otavio Salvador
2011-09-16 17:16       ` Paul Eggleton
2011-09-16 17:18         ` Otavio Salvador
2011-09-16 17:49           ` Paul Eggleton [this message]
2011-09-16 19:06             ` Otavio Salvador
2011-09-16 19:45               ` Mark Hatle
2011-09-16 20:51                 ` Otavio Salvador
2011-09-17  2:55                 ` Chris Larson
2011-09-17  9:06                   ` Koen Kooi
2011-09-17 12:05                     ` Otavio Salvador

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=201109161849.52647.paul.eggleton@linux.intel.com \
    --to=paul.eggleton@linux.intel.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=otavio@ossystems.com.br \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox