All of lore.kernel.org
 help / color / mirror / Atom feed
From: Khem Raj <raj.khem@gmail.com>
To: Daniel Dickinson <cshored@cshore.thecshore.com>,
	Gary Thomas <gary@mlbassoc.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: Hints on submitting to OE-Core
Date: Fri, 17 Mar 2017 19:17:36 -0700	[thread overview]
Message-ID: <c72438d4-adb6-0c1c-136c-eeae2ef3b8a8@gmail.com> (raw)
In-Reply-To: <20170317213807.377236b9@danielf.thecshore.com>


[-- Attachment #1.1: Type: text/plain, Size: 1724 bytes --]



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

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

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

>  
>>>
>>> Also, in the layers list I'll be asking about some build issues
>>> that I don't see with a manual bitbake, that look like missing
>>> preqs on a build server, unless the layer should be requesting
>>> installation of host files somewhere.  
>>
>> Some tools required by ${HOSTTOOLS} are missing:
>>    cpio chrpath gawk diffstat makeinfo
> 
> As mentioned these are openembedded.org autobuilds of submitted layers,
> so I was kind of hoping there were something I could change in the
> layer to make it work.  I guess the build host or chroot is missing
> packges.
> 
> Regards,
> 
> Daniel
> 


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 211 bytes --]

  reply	other threads:[~2017-03-18  2:17 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 [this message]
2017-03-18  2:36       ` Daniel Dickinson
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=c72438d4-adb6-0c1c-136c-eeae2ef3b8a8@gmail.com \
    --to=raj.khem@gmail.com \
    --cc=cshored@cshore.thecshore.com \
    --cc=gary@mlbassoc.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.