On Fri, Apr 13, 2018 at 6:01 AM Daniel Vetter <daniel.vet...@ffwll.ch> wrote:
> This tries to align with the X.org communities's long-standing > tradition of trying to be an inclusive community and handing out > commit rights fairly freely. > We also tend to not revoke commit rights for people no longer > regularly active in a given project, as long as they're still part of > the larger community. > Finally make sure that commit rights, like anything happening on fd.o > infrastructre, is subject to the fd.o's Code of Conduct. > v2: Point at MAINTAINERS for contact info (Daniel S.) > v3: > - Make it clear that commit rights are voluntary and that committers > need to acknowledge positively when they're nominated by someone > else (Keith). > - Encourage committers to drop their commit rights when they're no > longer active, and make it clear they'll get readded (Keith). > - Add a line that maintainers and committers should actively nominate > new committers (me). > v4: Typo (Petri). > v5: Typo (Sean). I don't remember this, but good for me! Current revision looks good, and I am supportive of applying it to drm-misc. Acked-by: Sean Paul <seanp...@chromium.org> > v6: Wording clarifications and spelling (Jani). > v7: Require an explicit commitment to the documented merge criteria > and rules, instead of just the implied one through the Code of Conduct > threat (Jani). > Acked-by: Alex Deucher <alexander.deuc...@amd.com> > Acked-by: Arkadiusz Hiler <arkadiusz.hi...@intel.com> > Acked-by: Daniel Stone <dan...@fooishbar.org> > Acked-by: Eric Anholt <e...@anholt.net> > Acked-by: Gustavo Padovan <gust...@padovan.org> > Acked-by: Petri Latvala <petri.latv...@intel.com> > Cc: Alex Deucher <alexander.deuc...@amd.com> > Cc: Arkadiusz Hiler <arkadiusz.hi...@intel.com> > Cc: Ben Widawsky <b...@bwidawsk.net> > Cc: Daniel Stone <dan...@fooishbar.org> > Cc: Dave Airlie <airl...@gmail.com> > Cc: Eric Anholt <e...@anholt.net> > Cc: Gustavo Padovan <gust...@padovan.org> > Cc: Jani Nikula <jani.nik...@intel.com> > Cc: Joonas Lahtinen <joonas.lahti...@linux.intel.com> > Cc: Keith Packard <kei...@keithp.com> > Cc: Kenneth Graunke <kenn...@whitecape.org> > Cc: Kristian H. Kristensen <hoegsb...@google.com> > Cc: Maarten Lankhorst <maarten.lankho...@linux.intel.com> > Cc: Petri Latvala <petri.latv...@intel.com> > Cc: Rodrigo Vivi <rodrigo.v...@intel.com> > Cc: Sean Paul <seanp...@chromium.org> > Reviewed-by: Keith Packard <kei...@keithp.com> > Signed-off-by: Daniel Vetter <daniel.vet...@intel.com> > --- > If you wonder about the wide distribution list for an igt patch: I'd > like to start a discussions about x.org community norms around commit > rights at large, at least for all the shared repos. I plan to propose > the same text for drm-misc and libdrm too, and hopefully others like > mesa/xserver/wayland would follow. > fd.o admins also plan to discuss this (and a pile of other topics and > hosting and code of conduct) with all projects, ideally this here > would end up as the starting point for establishing some community > norms. > -Daniel > --- > CONTRIBUTING | 48 ++++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 48 insertions(+) > diff --git a/CONTRIBUTING b/CONTRIBUTING > index 0180641be3aa..8a118134275c 100644 > --- a/CONTRIBUTING > +++ b/CONTRIBUTING > @@ -51,4 +51,52 @@ A short list of contribution guidelines: > - Changes to the testcases are automatically tested. Take the results into > account before merging. > +Commit rights > +------------- > + > +Commit rights will be granted to anyone who requests them and fulfills the > +below criteria: > + > +- Submitted a few (5-10 as a rule of thumb) non-trivial (not just simple > + spelling fixes and whitespace adjustment) patches that have been merged > + already. > + > +- Are actively participating on discussions about their work (on the mailing > + list or IRC). This should not be interpreted as a requirement to review other > + peoples patches but just make sure that patch submission isn't one-way > + communication. Cross-review is still highly encouraged. > + > +- Will be regularly contributing further patches. This includes regular > + contributors to other parts of the open source graphics stack who only > + do the oddball rare patch within igt itself. > + > +- Agrees to use their commit rights in accordance with the documented merge > + criteria, tools, and processes. > + > +Apply for an account (and any other account change requests) through > + > +https://www.freedesktop.org/wiki/AccountRequests/ > + > +and please ping the maintainers if your request is stuck. > + > +Committers are encouraged to request their commit rights get removed when they > +no longer contribute to the project. Commit rights will be reinstated when they > +come back to the project. > + > +Maintainers and committers should encourage contributors to request commit > +rights, especially junior contributors tend to underestimate their skills. > + > +Code of Conduct > +--------------- > + > +Please be aware the fd.o Code of Conduct also applies to igt: > + > +https://www.freedesktop.org/wiki/CodeOfConduct/ > + > +See the MAINTAINERS file for contact details of the igt maintainers. > + > +Abuse of commit rights, like engaging in commit fights or willfully pushing > +patches that violate the documented merge criteria, will also be handled through > +the Code of Conduct enforcement process. > + > Happy hacking! > -- > 2.16.2 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
