[Buildroot] [PATCH 1/1] Added local directory as soure of kernel code

Yann E. MORIN yann.morin.1998 at free.fr
Wed Feb 13 18:08:10 UTC 2013


Rafał, All,

On Wednesday 13 February 2013 Rafał Fabich wrote:
> On Tue, Feb 12, 2013 at 6:15 PM, Arnout Vandecappelle <arnout at mind.be> wrote:
> >
> >  One comment below, though.
> >
> >> +       default "local" if BR2_LINUX_KERNEL_CUSTOM_LOCAL
> >
> >
> >  This is a bit confusing, because the 'local' site method will force
> > LINUX_VERSION to "custom". So I would call it custom here as well.
> >
> 
> I would like to send a new version of this patch, should I create a
> new thread or use this one?

Create a new thread. Also, specify in the subject that this is version 2
of your patch, eg.:

    [PATH v2] Add local directory as soure of kernel code

('git send-email' helps you do that easily.)

And add the changes to your patch commit, after a line starting with
three dashes (git interpret this as an end-of-commit-log' separator:

---8<---
    linux: add option to use local directory as kernel source
    
    Add the option to use a local directory as the source for
    building the Linux kernel, which can be usefull during
    kernel development.
    
    ---
    Changes v1 -> v2
      - fixed foo bar  (Reviewer)
      - added baz buz
---8<---

This way, it is easy for the reviewer what to look at, and for the
maintainer as it knws a previous patch with the same subject is not
to be applied.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'


More information about the buildroot mailing list