From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Denys Dmytriyenko <denis@denix.org>, Marek Vasut <marex@denx.de>
Cc: openembedded-core@lists.openembedded.org,
openembedded-devel@lists.openembedded.org
Subject: Re: [PATCH] u-boot: Update to 2017.03 release
Date: Wed, 29 Mar 2017 23:16:07 +0100 [thread overview]
Message-ID: <1490825767.13980.333.camel@linuxfoundation.org> (raw)
In-Reply-To: <20170329215658.GP15664@denix.org>
On Wed, 2017-03-29 at 17:56 -0400, Denys Dmytriyenko wrote:
> On Mon, Mar 27, 2017 at 04:31:16PM +0200, Marek Vasut wrote:
> >
> > On 03/27/2017 04:25 PM, Richard Purdie wrote:
> > >
> > > On Mon, 2017-03-27 at 16:22 +0200, Marek Vasut wrote:
> > > >
> > > > Upgrade U-Boot to the latest version.
> > > Wrong list and how does this compare to Ovatio's patch?
> > I was not CCed on Otavio's patch :(
> Yeah, me neither, unfortunately - people just ignore maintainers file
> now.
>
> Good thing Ross copied me on the other discussion!
This raises a very valid point which is what any maintainers file is
and what it is not.
The one that exists in meta-yocto today has a very specific meaning. I
know there is discussion about moving it to OE-Core however if we do so
I want to by *really* *really* clear about what being listed there
means and does not mean.
As things stand right now, there is no requirement or advice in any of
our guides to cc "maintainers". People struggle to send patches as it
is and giving them more hoops to jump through doesn't seem like a
brilliant idea to me. I know we can create a file with all kinds of
path matching for files people care/don't care about but I worry this
will descend into complicated scripts and people still getting upset
when they were cc'd needlessly or weren't cc'd etc.
I also quite strongly dislike the "I own X, so only I can change it"
type view, to be the maintainers are people who help look after
specific areas in the absence of anyone else and help review patches
that come in. They certainly get significant influence on things but
the price for that is they also have to help fix issues and figure out
ways of helping others with issues in that area.
So please think very carefully about what you want this "maintainers"
file to be/not be as I worry its a huge can of worms we once tried in
OE before...
Cheers,
Richard
next prev parent reply other threads:[~2017-03-29 22:16 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20170327142231.21807-1-marex@denx.de>
[not found] ` <1490624701.13980.267.camel@linuxfoundation.org>
[not found] ` <5c037512-7bf1-8a3c-cb76-fda7230781a0@denx.de>
2017-03-29 21:56 ` [PATCH] u-boot: Update to 2017.03 release Denys Dmytriyenko
2017-03-29 22:16 ` Richard Purdie [this message]
2017-03-29 23:00 ` Denys Dmytriyenko
2017-03-30 8:21 ` Marek Vasut
2017-03-30 13:24 ` Denys Dmytriyenko
2017-03-30 14:12 ` Marek Vasut
2017-03-30 18:06 ` Denys Dmytriyenko
2017-03-30 19:12 ` Philip Balister
2017-03-30 19:26 ` Otavio Salvador
2017-03-31 11:55 ` Burton, Ross
2017-03-27 14:32 Marek Vasut
2017-03-28 11:07 ` Philip Balister
-- strict thread matches above, loose matches on Subject: below --
2017-03-26 20:08 Otavio Salvador
2017-03-26 21:14 ` Denys Dmytriyenko
2017-03-27 13:19 ` Otavio Salvador
2017-03-27 13:19 ` Burton, Ross
2017-03-27 13:28 ` Otavio Salvador
2017-03-27 15:47 ` Denys Dmytriyenko
2017-03-27 15:52 ` Burton, Ross
2017-03-27 17:49 ` Denys Dmytriyenko
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=1490825767.13980.333.camel@linuxfoundation.org \
--to=richard.purdie@linuxfoundation.org \
--cc=denis@denix.org \
--cc=marex@denx.de \
--cc=openembedded-core@lists.openembedded.org \
--cc=openembedded-devel@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox