On 2018-12-05 15:44:18, Jason Ekstrand wrote: > On Wed, Dec 5, 2018 at 5:32 PM Jordan Justen <jordan.l.jus...@intel.com> > wrote: > > -<h2 id="mailing">Mailing Patches</h2> > > +<h2 id="submit">Submitting Patches</h2> > > > > <p> > > -Patches should be sent to the mesa-dev mailing list for review: > > +Patches may be submitted to the Mesa project by > > +<a href="#mailing">email</a> or with a > > +GitLab <a href="#merge-request">merge request</a>. To prevent > > +duplicate code review, only use one method to submit your changes. > > +</p> > > > > Do we want to require a cover-letter to be sent to the ML? Ideally, we'd > just have a bot that makes one every time someone submits a MR and sends it > to the list. Maybe someone just needs to write that bot. > > > + > > +<h3 id="mailing">Mailing Patches</h3> > > + > > +<p> > > +Patches may be sent to the mesa-dev mailing list for review: > > <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev";> > > mesa-dev@lists.freedesktop.org</a>. > > When submitting a patch make sure to use > > @@ -217,8 +228,63 @@ disabled before sending your patches. (Note that you > > may need to contact > > your email administrator for this.) > > </p> > > > > +<h3 id="merge-request">GitLab Merge Requests</h3> > > + > > +<p> > > + <a href="https://gitlab.freedesktop.org/mesa/mesa";>GitLab</a> Merge > > + Requests (MR) can also be used to submit patches for Mesa. > > +</p> > > + > > +<p> > > + If the MR may have interest for most of the Mesa community, you can > > + send an email to the mesa-dev email list including a link to the MR. > > + Don't send the patch to mesa-dev, just the MR link.
Regarding the cover-letter, I put in this weasel worded sentence. Should it instead say this is required and that it should be a git format-patch generated cover letter? Or, should we drop it entirely and assume we'll get an automated way to send an email to the list whenever a new MR is opened? Relatedly, I think it might be possible to enable an irc channel to be notified about pushes and MR's. Not sure if it'd be a good idea, or maybe too noisy. -Jordan _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
