Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Cc: Scott Garman <scott.a.garman@intel.com>
Subject: Re: [yocto] RFC: Merging commits from into the main denzil branch
Date: Thu, 24 May 2012 17:54:41 +0100	[thread overview]
Message-ID: <1337878481.8248.154.camel@ted> (raw)
In-Reply-To: <22FC307C-7108-4DDC-9671-23AA109E964B@dominion.thruhere.net>

On Thu, 2012-05-24 at 18:04 +0200, Koen Kooi wrote:
> Op 24 mei 2012, om 17:46 heeft Scott Garman het volgende geschreven:
> 
> > On 05/24/2012 04:26 AM, Koen Kooi wrote:
> >> Op 24 mei 2012, om 00:00 heeft Scott Garman het volgende geschreven:
> >> 
> >>> Hello all,
> >>> 
> >>> As most of you know, I've been pulling commits into a pair of
> >>> sgarman/denzil-next branches which are intended to eventually
> >>> become the next Denzil point-release, 1.2.1.
> >>> 
> >>> oe-core based branch:
> >>> http://git.openembedded.org/openembedded-core-contrib/log/?h=sgarman/denzil-next
> >> 
> >>> 
> >> Would it be possible to do that in the main repo? That saves people
> >> from needing to add and fetch yet another repo.
> > 
> > I can't start with the main repo, because I don't have commit access to
> > it, and until I have done some significant testing with my commits, I
> > want to preserve the ability to rebase my branch.
> > 
> > However, now that I have some autobuilder resources to do more extensive
> > testing, what I'm proposing is to get my *tested* commits moved from
> > sgarman/denzil-next into the denzil branch after review from the
> > community and a merge that Richard will do.
> > 
> > That means the denzil branch will lag my sgarman/denzil-next branch by a
> > certain amount of time, which I intend to keep down to less than a week.

What we could do is give you direct access to the denzil-next branch in
the main repository which would be a happy medium between these two
states since you could still rebase it.
 
> >>> So now to address issue #2. The goal is to incorporate community
> >>> feedback, so I'm looking to get ACKs or NAKs for these commits
> >>> before they go into the main denzil branch.
> >>> 
> >>> My proposal is to send denzil pull requests to the appropriate
> >>> mailing lists, and Richard can merge them into the main repo denzil
> >>> branches once they've received review by the community.
> >>> 
> >>> My goal is to send these pull requests about once per week, once
> >>> I've managed to get a green build out of the Yocto autobuilder for
> >>> my contrib branch.
> >>> 
> >>> What do folks think about this? Now's your opportunity to offer
> >>> feedback and influence this process.
> >> 
> >> You're missing a process for people to propose commits for the denzil
> >> branch. How about this:
> >> 
> >> If you want to propose a commit from master cherry-pick it into a
> >> local denzil branch and then send if for review to the oe-core list
> >> with the [for-denzil] subject prefix. Scott will take care of putting
> >> it in the pull requests based on the feedback the patch gets.
> > 
> > I can work with that. But keep in mind that if the request is not
> associated with a current bug marked for 1.2.1 in the Yocto bugzilla,
> I will need to see strong community support for including it (aka,
> multiple ACKs). This is needed to avoid scope creep in the
> point-release.
>
> I think you're confusing oe-core with yocto

I think some process for showing support for changes is good and the
Ack's seem as good as any other.

Cheers,

Richard




      reply	other threads:[~2012-05-24 17:05 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-23 22:00 RFC: Merging commits from into the main denzil branch Scott Garman
2012-05-24 11:26 ` [yocto] " Koen Kooi
2012-05-24 15:46   ` Scott Garman
2012-05-24 16:04     ` Koen Kooi
2012-05-24 16:54       ` Richard Purdie [this message]

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=1337878481.8248.154.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=scott.a.garman@intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox