Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/2 v3] odroid-scripts: New Package.
Date: Sun, 24 Jul 2016 15:33:20 +0200	[thread overview]
Message-ID: <20160724153320.1329f844@free-electrons.com> (raw)
In-Reply-To: <trinity-ae8b0b55-8d97-4c34-9726-f0a50212c8e1-1469366784462@3capp-mailcom-bs05>

Hello,

On Sun, 24 Jul 2016 15:26:24 +0200, daggs wrote:

> > The changelog should not be part of the commit log.
> >   
> > > 
> > > Signed-off-by: Dagg Stompler <daggs@gmx.com>
> > > ---  
> > 
> > It should be here.  
> 
> I always seems to forget were to place it.... :-/

It's pretty simple:

"""
foo: new package

Blabla what this new package is all about. Blabla.

Signed-off-by: John Doe
---
Changes since v2:
 ...
"""

Everything that is *above* the "---" sign gets kept in the Git history
as the commit log. Everything that is *below" the "---" sign is ignored
when the patch is applied.

Since the changelog is describing what was changed between iterations
of the patch, we don't want the changelog to be committed forever in
the Git history, so it should be below the "---" sign.

When doing a commit, you can simply introduce a "---" manually below
your Signed-off-by and write your changelog here.


> this resulted im my serial usb stop working, e.g. I didn't know where I was in the boot nor couldn't log in into the system.
> 
> without both of the changes, the system boots ok.

OK, makes sense. Thanks for explaining.

> > Other than that, looks good. Can you respin a new version?  
> 
> maybe later today.

Sure, or another day, there's no hurry.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

      reply	other threads:[~2016-07-24 13:33 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-24  5:56 [Buildroot] [PATCH 1/2 v3] odroid-scripts: New Package Dagg Stompler
2016-07-24  5:56 ` [Buildroot] [PATCH 2/2 v4] odroid-mali: " Dagg Stompler
2016-07-24 12:44   ` Thomas Petazzoni
2016-07-24 13:29     ` daggs
2016-07-24 13:34       ` Thomas Petazzoni
2016-07-24 12:40 ` [Buildroot] [PATCH 1/2 v3] odroid-scripts: " Thomas Petazzoni
2016-07-24 13:26   ` daggs
2016-07-24 13:33     ` Thomas Petazzoni [this message]

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=20160724153320.1329f844@free-electrons.com \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=buildroot@busybox.net \
    /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