Hello,
This fixes a small typo.
Regards, Roman
Hi Roman,
On Wed, Jan 31, 2007 at 03:02:21PM -0600, Roman Kononov wrote:
This fixes a small typo.
Thanks for the patches!
Please have a look at http://linuxbios.org/Development_Guidelines and specifically the "How to contribute" section, so your patches can make it into svn faster.
//Peter
This fixes a small typo.
Signed-off-by: Roman Kononov kononov195-lbl@yahoo.com ---
On 01/31/2007 03:19 PM, Peter Stuge wrote:
Please have a look at http://linuxbios.org/Development_Guidelines
Thank you
On 1/31/07, Peter Stuge stuge-linuxbios@cdy.org wrote:
Please have a look at http://linuxbios.org/Development_Guidelines and specifically the "How to contribute" section, so your patches can make it into svn faster.
apropos this comment. Could someone add a little clarification about what happens once an Acked-by: is added?
I would assume that if someone with commit rights adds an acked-by:, they take responsibility for the commit. But that is not spelled out.
Thanks
ron
On Wed, Jan 31, 2007 at 04:25:27PM -0600, Roman Kononov wrote:
This fixes a small typo.
Signed-off-by: Roman Kononov kononov195-lbl@yahoo.com
Committed, thanks.
Uwe.
On Wed, Jan 31, 2007 at 03:42:47PM -0700, ron minnich wrote:
On 1/31/07, Peter Stuge stuge-linuxbios@cdy.org wrote:
Please have a look at http://linuxbios.org/Development_Guidelines and specifically the "How to contribute" section, so your patches can make it into svn faster.
apropos this comment. Could someone add a little clarification about what happens once an Acked-by: is added?
I would assume that if someone with commit rights adds an acked-by:, they take responsibility for the commit. But that is not spelled out.
Yes, basically. The Signed-off-by is legally important to have a clear statement from the contributors that they can and want to legally contribute the code under the GPL etc., while the Acked-by is just a mechanism to somewhat enforce peer review of the patches.
One Acked-by is the minimum which is required to be able to commit a patch (the more the better, of course). This means at least two people looked at the patch and didn't notice any glaring bugs/problems (the patch submitter + the one who sent the Acked-by).
People without svn commit access are welcome to review patches and send Acked-by's too, but obviously someone _with_ commit access has to do the actual commit then.
Cheers, Uwe.