On 06/03/15 18:14, Ard Biesheuvel wrote:

> Thanks, pushed as r17554

Congrats to your first (am I right?) SVN commit!

A small tip (feel free to ignore it): since for edk2 patches, the
tianocore contribution agreement is needed, it's useful to set up a
commit message template. Something like:

[commit]
        template = /home/ardb/misc/tianocore.template

where the referenced file would say:

--------------


Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ard Biesheuvel <ard.biesheu...@linaro.org>
-----------------

The "trick" I'm proposing here is that you include your S-o-b with the
template as well, since you "should" have a template anyway. (Or else,
pass --signoff to "git-commit".)

Additionally, ask git-format-patch not to generate your S-o-b's on the fly:

[format]
        signoff = false

Your posted patches will look the same, but the difference will be that
your commit messages themselves will include your signoffs; the S-o-b
tags won't be generated *only* when formatting the patches.

Why is this good? Because the resultant signoffs-in-commits will be
consistent with patches that you apply from others (with git-am), and
because this way you can add my R-b tag *after* your S-o-b, via rebasing.

Just my two cents (in a wall of text, sorry), feel free to ignore.

Thanks
Laszlo

------------------------------------------------------------------------------
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to