All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel Dickinson <cshored@cshore.thecshore.com>
To: openembedded-core@lists.openembedded.org
Subject: Re: Hints on submitting to OE-Core
Date: Fri, 17 Mar 2017 22:36:48 -0400	[thread overview]
Message-ID: <20170317223035.697b8832@danielf.thecshore.com> (raw)
In-Reply-To: <c72438d4-adb6-0c1c-136c-eeae2ef3b8a8@gmail.com>

On Fri, 17 Mar 2017 19:17:36 -0700
Khem Raj <raj.khem@gmail.com> wrote:

> On 3/17/17 6:38 PM, Daniel Dickinson wrote:
> > On Sat, 18 Mar 2017 02:21:30 +0100
> > Gary Thomas <gary@mlbassoc.com> wrote:
> >   
> >>
> >> Have you read this?
> >>    http://www.openembedded.org/wiki/How_to_submit_a_patch_to_OpenEmbedded  
> > 
> > I hadn't read in detail yet (I've seen similar before and from
> > skimming, it's about mechanics of the process), but I was looking
> > more in terms of coding and/or style guidelines for the meat of the
> > patch, and/or hints/tips on getting things accepted from the point
> > of view of things other than mechanics of patch submission.  
> 
> If you read through that link Gary pointed, there are links to two
> other documents one of style and other one patch message guidelines
> 

Ah, missed that on initial perusal, thank you.

> there is styling scripts for metadata files, you could use that
> https://github.com/openembedded/meta-openembedded/blob/master/contrib/oe-stylize.py


Will use thatt.

> As of meat of patch, if its fixing something and you are articulate to
> explain it in your commit message and in general review cycle, you
> have a better chance of getting it accepted, but you have to start
> submitting

I'm kind of hoping to get some feedback on the meta-runit-init, and
meta-earlyinit layers I've submitted to the layers listing (and to find
things I've missed) (perhaps as well as meta-cshored, but I don't think
there is as much there that's as likely to be generally useful) before
looking at working on getting them integrated into core.  Of course it
helps if the layers listing doesn't show builds failing (in this case
not because of an error on my part).

Regards,

Daniel


  reply	other threads:[~2017-03-18  2:36 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-17 23:55 Hints on submitting to OE-Core Daniel Dickinson
2017-03-18  1:21 ` Gary Thomas
2017-03-18  1:38   ` Daniel Dickinson
2017-03-18  2:17     ` Khem Raj
2017-03-18  2:36       ` Daniel Dickinson [this message]
2017-03-18  8:42   ` akuster808
2017-03-18 17:17     ` Daniel Dickinson
2017-03-18 19:05     ` Daniel Dickinson

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=20170317223035.697b8832@danielf.thecshore.com \
    --to=cshored@cshore.thecshore.com \
    --cc=openembedded-core@lists.openembedded.org \
    /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.