From: Trevor Woerner <twoerner@gmail.com>
To: Paul Eggleton <paul.eggleton@linux.intel.com>,
openembedded-core@lists.openembedded.org,
openembedded-devel@lists.openembedded.org,
openembedded-architecture@lists.openembedded.org
Subject: Re: Patchwork & patch handling improvements
Date: Mon, 30 Nov 2015 10:19:35 -0500 [thread overview]
Message-ID: <565C6907.3020501@gmail.com> (raw)
In-Reply-To: <1633798.xAxFJSpIzb@peggleto-mobl.ger.corp.intel.com>
On 11/26/15 16:00, Paul Eggleton wrote:
> I'm also
> trying to ensure that the patch validation is generic enough so it can live in
> OE-Core, and thus we can easily update and refine it over time in line with the
> code itself as well as encourage submitters to use the script on their own
> changes before sending.
This all sounds like an improvement and is therefore a step in the right
direction :-)
A while back I had the idea of "porting" the kernel's "checkpatch.pl" to
The Yocto Project (it was around the same time that I was trying to
float the whole "Maintainers File" idea too, since I was also trying to
re-purpose "get-maintainer.pl" as well). About one minute into that
effort I realized the existing *.bb files were all over the place in
terms of the order of statements and the order of the blocks of
statements. At that time I found one recipe style guide from OE, and
another one from The Yocto Project, each of which described a slightly
different preference. So I asked on the mailing list and quickly
discovered that both groups prefer a different style.
I'm not saying this job isn't worth doing, but I am pointing out there's
the potential for feathers to be ruffled on both sides if someone tries
to produce a definitive style guide for recipe files and then enforces
it in an automated way. Since it is the OpenEmbedded Project's job to
provide the recipes for The Yocto Project, I'm guessing this question
needs to be decided by them? If that sounds reasonable, then maybe The
Yocto Project needs to acquiesce to OE's decision?
Instead of cross-posting, maybe this would be a good email for the new
architecture list (CC'ed)?
next prev parent reply other threads:[~2015-11-30 15:19 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-26 21:00 Patchwork & patch handling improvements Paul Eggleton
2015-11-26 21:12 ` Burton, Ross
2015-11-27 17:15 ` akuster808
2015-11-29 20:06 ` Paul Eggleton
2015-11-30 13:56 ` Koen Kooi
2015-11-30 15:19 ` Trevor Woerner [this message]
2015-11-30 18:49 ` Paul Eggleton
2015-12-01 10:47 ` Martin Jansa
2015-12-02 3:01 ` Paul Eggleton
2015-12-02 8:17 ` Martin Jansa
2015-12-02 10:54 ` Barros Pena, Belen
2015-12-02 10:59 ` Barros Pena, Belen
2015-12-02 18:04 ` Martin Jansa
2015-12-02 18:43 ` [Openembedded-architecture] " Burton, Ross
2015-12-02 21:58 ` Tim Orling
2015-12-03 11:43 ` Barros Pena, Belen
2015-12-03 12:51 ` Burton, Ross
2015-12-03 14:05 ` Martin Jansa
2015-12-03 14:43 ` Barros Pena, Belen
2015-12-03 11:38 ` Barros Pena, Belen
2015-12-02 8:44 ` Richard Purdie
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=565C6907.3020501@gmail.com \
--to=twoerner@gmail.com \
--cc=openembedded-architecture@lists.openembedded.org \
--cc=openembedded-core@lists.openembedded.org \
--cc=openembedded-devel@lists.openembedded.org \
--cc=paul.eggleton@linux.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