Delegate Vs Reviewer
Mauro Carvalho Chehab
mchehab at osg.samsung.com
Wed Nov 26 23:14:19 AEDT 2014
Em Wed, 26 Nov 2014 13:42:21 +0800
Jeremy Kerr <jk at ozlabs.org> escreveu:
> Hi all,
> > On Mon, Nov 24, 2014 at 05:35:11PM +0000, Damien Lespiau wrote:
> >> I'd like to be able to assign reviewers to patches. That seems very
> >> close to what a delegate is today. Would anyone have objections if I
> >> just use Delegate as Reviewer?
> > I'm honestly not sure what Delegate is intended for. It seems like its
> > purpose is open to your interpretation.
> One example: on the powerpc list, we have an overall maintainer, and
> several sub-maintainers, who are generally responsible for their own
> subsystem or set of hardware.
> If a patch for one of the sub-maintiners' areas appears, it can be
> delegated to the appropriate person to review, test, and/or merge. The
> change is then included in the next merge-request from the
> sub-maintainer to the main maintainer.
We do the same at the media subsystem. Laurent even wrote some patches
for patchwork to do patch auto-delegation based on the paths inside
the diffstat, using regex expressions.
Not sure if he sent those patches upstream.
> In this case, patch review is only one of the tasks of the delegate.
> Patchwork mailing list
> Patchwork at lists.ozlabs.org
More information about the Patchwork