review tools (was Re: Code review required for commits - formal Team vote.)

David Disseldorp ddiss at suse.de
Wed Oct 17 09:32:16 MDT 2012


On Wed, 17 Oct 2012 11:12:29 -0400
simo <idra at samba.org> wrote:

> > For an look at patchwork in action, see:
> > https://patchwork.kernel.org/project/linux-sh/list/  
> 
> Sounds nice and it seem that thanks to the git hook it should work even
> if it is not used by everybody, meaning we could set it up now and just
> let it run and let it be used by whoever wants to use it.
> Is that right ?

Yes, that's correct.

> If so I can volunteer to use it and even to set it up, heck even host
> it.

Great, I volunteer to use it too. Let me know if I can do anything
regarding setup.
Until then I recommend developers get acquainted with git send-email.

> However I am afraid we may need to create a separate mailing list for
> reviews/patches in order to avoid drowning other technical discussions ?

"PATCH" is easy to filter out, but I'm not too concerned either way.

Cheers, David


More information about the samba-technical mailing list