All of lore.kernel.org
 help / color / mirror / Atom feed
From: York Sun <york.sun@nxp.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] u-boot release and feature set
Date: Fri, 31 Aug 2018 16:45:41 +0000	[thread overview]
Message-ID: <DB7PR04MB42980E3E11A2A549381EB6669A0F0@DB7PR04MB4298.eurprd04.prod.outlook.com> (raw)
In-Reply-To: 20180831163037.GU26633@bill-the-cat

On 08/31/2018 09:30 AM, Tom Rini wrote:
> On Fri, Aug 31, 2018 at 03:59:59PM +0000, York Sun wrote:
>> On 08/31/2018 08:49 AM, Tom Rini wrote:
>>> I really would appreciate the level of detail Linus' tree gets in signed
>>> commit messages being in the U-Boot tree too.  No, it's not perfect and
>>> it does need someone to make it more digestible, but it's a good
>>> starting point.  And the people that do send me signed tags do a good
>>> job too I think.
>>>
>>
>> If you prefer to have signed tags, we can start to do this. Would you
>> write some guideline on the format, and what information should be
>> included in the annotated tags, how often do we tag?I guess you will use
>> a script to extract information, unless you want to read all of them.
>  
> I will admit that I had to manually find this email link as perhaps my
> subject at the time was not the best:
> https://lists.denx.de/pipermail/u-boot/2018-June/330610.html
> 
> If information is in the merge commit then it's a lot easier to do
> anything more on top of that.  Thanks!
> 

Tom,

I have been trying to avoid merge commit. I do my best to rebase my tree
before requesting a PR so it would be easier for you to merge.

Using signed tag is not an issue. But you still don't have the important
changes separated from trivial changes, unless this information is
included in the message. For example, if I request a pull with 50
commits, in which 40 commits only deal with default environmental
variables. The other 10 commits add a new driver which will be shared
for new platforms. I consider the useful information in this PR is the
new driver. If this description is in the tag message, it would be
easier for you to gather and put into a release note. So having a
predefined format will help you to parse the message. I don't think you
prefer to read all tags with your eyes.

York

  reply	other threads:[~2018-08-31 16:45 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-08-28  6:48 [U-Boot] u-boot release and feature set Prabhakar Kushwaha
2018-08-29  1:57 ` Tom Rini
2018-08-29  3:11   ` Prabhakar Kushwaha
2018-08-31 15:46     ` York Sun
2018-08-31 15:49       ` Tom Rini
2018-08-31 15:59         ` York Sun
2018-08-31 16:30           ` Tom Rini
2018-08-31 16:45             ` York Sun [this message]
2018-08-31 16:58               ` Tom Rini

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=DB7PR04MB42980E3E11A2A549381EB6669A0F0@DB7PR04MB4298.eurprd04.prod.outlook.com \
    --to=york.sun@nxp.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.