Re: [PATCH] SubmittingPatches: clarify SOB tag usage when evolvingsubmissions

From: Dan Carpenter
Date: Wed Aug 15 2012 - 04:23:24 EST


I think it would be nice to have another tag for people who fix bugs
in the original patch. The Reviewed-by tag implies approval of the
whole patch and anyway reviewers don't normally comment unless they
see a bug. Maybe something like:

Contributor: Your Name <email@xxxxxxxxxxx>

So the tags for developers would be:

Signed-off-by: The patch went through you. Legal responsibility.
Acked-by: You know what you are talking about and approve.
Reviewed-by: You reviewed the patch and approve.
Contributor: You noticed or fixed a bug in the patch.

regards,
dan carpenter
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/