All of lore.kernel.org
 help / color / mirror / Atom feed
From: Graeme Russ <graeme.russ@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] OMAP3 SPL: merge window question
Date: Mon, 18 Jul 2011 19:39:40 +1000	[thread overview]
Message-ID: <4E23FF5C.7060600@gmail.com> (raw)
In-Reply-To: <4E23F307.7000105@aribaud.net>

Hi Simon, Albert

On 18/07/11 18:47, Albert ARIBAUD wrote:
> Le 18/07/2011 10:29, Simon Schwarz a ?crit :
>> Dear Wolfgang Denk, Sandeep Paulraj and Albert Aribaud,

[snip]

> Personally, when a patch was initially submitted before the end of the 
> merge window, I will accept subsequent versions of it and apply them 
> even if they are submitted after the merge window has closed.

Provided they are either a simple rebase or an update due to mailing list
comments or a correction. After the merge window closes, do not use the
existing patches posted inside the merge window to sneak in new features

> Only, be sure to follow *all* rules for submitting new versions of a 
> patch set as laid out in 
> <http://www.denx.de/wiki/view/U-Boot/Patches#Sending_updated_patch_versions>
> . Particularly:
> 
> - *always* provide history for *each* patch
> 
> - *always* provide history *below* the commit message delimiter ('---')
> 
> - *always* post version N of a patch "in-reply-to" version N-1.
> 
> - *never* post a patch set twice under the same version. Even if you 
> forgot some trivial thing like signed-off-by, repost with a *bumped up* 
> version number (and provide updated history)

And if the patch is a part of a series, update the version of *all* patches
in the series (with a 'no changes' comment for any individual patches in
the series which have not been update). Nothing worse than a series being a
mix of various versions.

> Be warned that not respecting these rules will result in the patch set 
> being NAKed (and, in turn, will force you to re-post a Vn+1 patch...)


Regards,

Graeme

  reply	other threads:[~2011-07-18  9:39 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-18  8:29 [U-Boot] OMAP3 SPL: merge window question Simon Schwarz
2011-07-18  8:47 ` Albert ARIBAUD
2011-07-18  9:39   ` Graeme Russ [this message]
2011-07-18 12:03     ` Wolfgang Denk
2011-07-18 12:11       ` Graeme Russ
2011-07-18 12:29         ` Wolfgang Denk
2011-07-18 13:26           ` Holger Brunck
2011-07-18 11:31   ` Simon Schwarz
2011-07-18 11:59     ` Graeme Russ
2011-07-18 12:11     ` Albert ARIBAUD
2011-07-18 12:30       ` Wolfgang Denk
2011-07-18 14:19         ` Daniel Schwierzeck
2011-07-18 14:54           ` Aneesh V
2011-07-18 15:15             ` Daniel Schwierzeck
2011-07-18 16:07           ` Wolfgang Denk
2011-07-26 12:46 ` Wolfgang Denk

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=4E23FF5C.7060600@gmail.com \
    --to=graeme.russ@gmail.com \
    --cc=u-boot@lists.denx.de \
    /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.