[yocto] [OE-core] Git commit process question.

akuster808 akuster808 at gmail.com
Mon Apr 1 16:20:41 PDT 2019



On 4/1/19 4:02 PM, Richard Purdie wrote:
> On Mon, 2019-04-01 at 15:33 -0700, akuster808 wrote:
>> Hello,
>>
>> I have noticed a large number of git commits with no header
>> information being accepted.
> Can you be more specific about what "no header information" means? You
> mean a shortlog and no full log message?
Commits with just a "subject" and signoff. No additional information

We tend to reference back to how the kernel does things.

https://www.kernel.org/doc/html/latest/process/submitting-patches.html
These two sections in particular.


    2) Describe your changes

Describe your problem. Whether your patch is a one-line bug fix or 5000
lines of a new feature, there must be an underlying problem that
motivated you to do this work. Convince the reviewer that there is a
problem worth fixing and that it makes sense for them to read past the
first paragraph.


along with this section.


    14) The canonical patch format

This section describes how the patch itself should be formatted. Note
that, if you have your patches stored in a |git| repository, proper
patch formatting can be had with |git format-patch|. The tools cannot
create the necessary text, though, so read the instructions below anyway.

The canonical patch subject line is:

Subject: [PATCH 001/123] subsystem: summary phrase

The canonical patch message body contains the following:

      * A |from| line specifying the patch author, followed by an empty
        line (only needed if the person sending the patch is not the
        author).
      * The body of the explanation, line wrapped at 75 columns, which
        will be copied to the permanent changelog to describe this patch.
      * An empty line.
      * The |Signed-off-by:| lines, described above, which will also go
        in the changelog.
      * A marker line containing simply |---|.
      * Any additional comments not suitable for the changelog.
      * The actual patch (|diff| output).


    - Armin

>
> Cheers,
>
> Richard
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.yoctoproject.org/pipermail/yocto/attachments/20190401/7f63be67/attachment.html>


More information about the yocto mailing list