Em Wed, 27 Nov 2024 10:39:48 +0100
Mauro Carvalho Chehab <mchehab+huawei@xxxxxxxxxx> escreveu:
> > This workflow doesn't apply to patch submitters who are not allowed to
> > send pull requests and who don't have direct commit access. I thought
> > these submitters are the main audience of this document. In that case, I
> > think moving the next section that explains the e-mail workflow before
> > the "Media development workflow" section (which should likely be renamed
> > to make it clear that it is about merging patches, not developing them)
> > would be best. The "Review Cadence" section could also be folded in
> > there, to give a full view of what a submitter can expect.
> >
> > This would also have the advantage of introducing the linuvtv.org
> > patchwork instance, which you reference above. Documents are more
> > readable when they introduce concepts first before using them.
>
> Will try to do such change at v2.
Actually, both workflows (a) and (b) apply to the ones that can't
send pull requests or push at media-committers.git:
---
a. Normal workflow: patches are handled by subsystem maintainers::
+------+ +---------+ +-------+ +-----------------------+ +---------+
|e-mail|-->|patchwork|-->|pull |-->|maintainers merge |-->|media.git|
+------+ +---------+ |request| |in media-committers.git| +---------+
+-------+ +-----------------------+
For this workflow, pull requests can be generated by a committer,
a previous committer, subsystem maintainers or by a couple of trusted
long-time contributors. If you are not in such group, please don't submit
pull requests, as they will likely be ignored.
b. Committers' workflow: patches are handled by media committers::
+------+ +---------+ +--------------------+ +-----------+ +---------+
|e-mail|-->|patchwork|-->|committers merge at |-->|maintainers|-->|media.git|
+------+ +---------+ |media-committers.git| |approval | +---------+
+--------------------+ +-----------+
---
No matter who sent an e-mail, this will be picked by patchwork and either
be part of a PR or a MR, depending on who picked it.
Thanks,
Mauro
