[Sugar-devel] Acked-by vs Reviewed-by

Daniel Narvaez dwnarvaez at gmail.com
Wed Apr 3 16:20:43 EDT 2013


Ok I see how you can read
https://www.kernel.org/doc/Documentation/SubmittingPatches that way.

Though, man, the whole document feels so unnecessarily complicated and
unclear if applied to Sugar. If we keep using these tags I think we should
write a simpler explanation which applies to our project.


On 3 April 2013 22:11, Gonzalo Odiard <gonzalo at laptop.org> wrote:

> I think only the maintainer can Ack (and imply a review)
> and anybody else can add a Reviewed-by
>
> Gonzalo
>
> On Wed, Apr 3, 2013 at 5:07 PM, Daniel Narvaez <dwnarvaez at gmail.com>wrote:
>
>> Hi,
>>
>> it seems that most of our patches should have a Reviewed-by tag, on the
>> contrary I see Acked-by used most of the time (at the top of the
>> sugar-toolkit-gtk3 log at least).
>>
>> Am I missing something?
>>
>> --
>> Daniel Narvaez
>>
>> _______________________________________________
>> Sugar-devel mailing list
>> Sugar-devel at lists.sugarlabs.org
>> http://lists.sugarlabs.org/listinfo/sugar-devel
>>
>>
>


-- 
Daniel Narvaez
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sugarlabs.org/archive/sugar-devel/attachments/20130403/a849cb7f/attachment.html>


More information about the Sugar-devel mailing list