Re: Switching SCM to git and commit/review policy

[Koen Kooi <k.kooi@student.utwente.nl>]

pHilipp Zabel wrote:

>>> Personally, I'd like to see every patch to OE being sent to and
>>> reviewed on the mailing list.
>> I'd like to see that every commit has at least 2 SOBs, I care less on how
>> they get there. Having the review out in the open should be the end goal,
>> though.
>
>  From Documentation/SubmittingPatches:
> "The Signed-off-by: tag indicates that the signer was involved in the
> development of the patch, or that he/she was in the patch's delivery path."
>
> In the kernel Signed-off-by is primarily used to mark the way a patch
> took into the kernel. If we do this, maybe we should use the same
> nomenclature and have Acked-by for statements of approval. (And
> eventually Tested-by/Reviewed-by, too?)

That is probably a good idea.

regards,

Koen