All of lore.kernel.org
 help / color / mirror / Atom feed
From: James Bottomley <James.Bottomley@HansenPartnership.com>
To: Jonathan Corbet <corbet@lwn.net>, Josh Poimboeuf <jpoimboe@redhat.com>
Cc: ksummit-discuss@lists.linuxfoundation.org
Subject: Re: [Ksummit-discuss] Maintainer's Summit Agenda Planning
Date: Fri, 06 Oct 2017 09:56:44 -0700	[thread overview]
Message-ID: <1507309004.3104.17.camel@HansenPartnership.com> (raw)
In-Reply-To: <20171006103259.78ab2508@lwn.net>

On Fri, 2017-10-06 at 10:32 -0600, Jonathan Corbet wrote:
> On Fri, 6 Oct 2017 11:26:21 -0500
> Josh Poimboeuf <jpoimboe@redhat.com> wrote:
> 
> > 
> > I think it would be a good idea to have a Maintainer's Guide which
> > tries to document a lot of this knowledge.  It would help new
> > maintainers learn the ropes, and would also help drive consensus
> > for maintainer's best practices.  It could document the typical
> > processes of a maintainer, and policy guidelines like some of the
> > above topics.
> 
> Strangely enough, this is a conversation that has been popping up in
> other contexts too.  We may see an initial attempt before too long.
> 
> The tricky part, of course, is finding a way to document the
> consensus on best practices without trying to "drive" it too hard.
> 
> My own thought is that a good starting place might be a "how to avoid
> getting your pull request flamed" document, since there is some
> semblance of a consensus there and it's a place where people often
> make mistakes.

Actually, I'd argue this is the most arcane area.  Accepting a pull
request represents the expression of a trust relationship and it's not
entirely well documented how to form that.  The mechanics of what
should be in it and how it should be split vary by puller.  For
instance, Linus' requirements are reasonably well documented in git-
request-pull, but other trees have varying requirements.  Why he might
flame you varies from too many merge window patches in a non-merge
window to to many merge points in the tree or "unclean" history.

James

  parent reply	other threads:[~2017-10-06 16:56 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-05 19:20 [Ksummit-discuss] Maintainer's Summit Agenda Planning Theodore Ts'o
2017-10-05 20:13 ` Rafael J. Wysocki
2017-10-05 21:55 ` Jiri Kosina
2017-10-06 14:59 ` Takashi Iwai
2017-10-06 15:27 ` James Bottomley
2017-10-06 16:26   ` Josh Poimboeuf
2017-10-06 16:32     ` Jonathan Corbet
2017-10-06 16:51       ` Josh Poimboeuf
2017-10-06 16:56       ` James Bottomley [this message]
2017-10-06 17:16         ` Josh Poimboeuf
2017-10-06 20:11       ` Linus Walleij
2017-10-09  8:13   ` Mark Brown
2017-10-09 15:54   ` Jiri Kosina
2017-10-09 16:37     ` James Bottomley
2017-10-09 16:47       ` Joe Perches
2017-10-09 16:49       ` Julia Lawall
2017-10-09 16:56         ` James Bottomley
2017-10-09 17:04           ` Joe Perches
2017-10-11 18:51           ` Jani Nikula
2017-10-12 10:03             ` Daniel Vetter
2017-10-16 14:12             ` James Bottomley
2017-10-16 14:25               ` Jani Nikula
2017-10-16 16:07                 ` Joe Perches
2017-10-17  8:34                   ` Jani Nikula
2017-10-18  1:27                     ` Joe Perches
2017-10-18 10:41                       ` Jani Nikula
2017-10-16 18:52               ` Mark Brown
2017-10-10  8:53       ` Jiri Kosina
2017-10-24 23:03   ` Kees Cook
2017-10-24 23:41     ` Joe Perches
2017-10-25  0:54       ` Kees Cook
2017-10-25  4:21         ` Julia Lawall
2017-10-25  4:29           ` Joe Perches
2017-10-25  4:36             ` Julia Lawall
2017-10-25  6:05         ` Martin K. Petersen
2017-10-25  6:55           ` Kees Cook
2017-10-25  7:34             ` Martin K. Petersen
2017-10-25  6:45         ` Frank Rowand
2017-10-25  7:56         ` Mark Brown
2017-10-25  9:39         ` Laurent Pinchart
2017-10-31 19:19         ` Rob Herring
2017-10-31 19:28           ` Kees Cook

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=1507309004.3104.17.camel@HansenPartnership.com \
    --to=james.bottomley@hansenpartnership.com \
    --cc=corbet@lwn.net \
    --cc=jpoimboe@redhat.com \
    --cc=ksummit-discuss@lists.linuxfoundation.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.