Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Phil Blundell <philb@gnu.org>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [RFC] Suggestion of minor change to patch submission policy re: long descriptions in commit headers
Date: Wed, 14 Sep 2011 10:30:57 +0100	[thread overview]
Message-ID: <1315992658.2519.137.camel@phil-desktop> (raw)
In-Reply-To: <1315961204.2252.32.camel@scimitar>

On Tue, 2011-09-13 at 17:46 -0700, Joshua Lock wrote:
> Whilst I intend to rectify the latter I'd like to propose we change the
> former such that the defect information is at the end of the commit
> message.
> 
> I believe this is more suitable for the project because the defect
> information and its relevance should be summarised in the long
> description, and therefore the defect id and link to the defect tracker
> are supplemental information for interested readers.
> 
> IMHO this supplementary nature should lead us to request submitters
> provide defect information after the long description.

Agreed, I think this would be something of an improvement (and indeed,
from a look at the git log it appears that some submitters are already
doing this).  Although it isn't a very big deal, I do find it slightly
irritating to have the first line of the long checkin message be
something that is essentially noise.

Possibly even better would be to think up a way to encode the defect
information, in some machine-readable form, as part of the
pseudo-header.  That would make it straightforward for folks who care
particularly about the defect information for (say) yocto to filter it
into some more prominent location.

p.





  parent reply	other threads:[~2011-09-14  9:36 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-14  0:46 [RFC] Suggestion of minor change to patch submission policy re: long descriptions in commit headers Joshua Lock
2011-09-14  0:58 ` Mark Hatle
2011-09-14  6:54   ` Saul Wold
2011-09-14  9:30 ` Phil Blundell [this message]
2011-09-16 17:03   ` 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=1315992658.2519.137.camel@phil-desktop \
    --to=philb@gnu.org \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox