<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Wed, Dec 5, 2018 at 7:05 PM Jordan Justen <<a href="mailto:jordan.l.justen@intel.com">jordan.l.justen@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 2018-12-05 15:44:18, Jason Ekstrand wrote:<br>
> On Wed, Dec 5, 2018 at 5:32 PM Jordan Justen <<a href="mailto:jordan.l.justen@intel.com" target="_blank">jordan.l.justen@intel.com</a>><br>
> wrote:<br>
> > -<h2 id="mailing">Mailing Patches</h2><br>
> > +<h2 id="submit">Submitting Patches</h2><br>
> ><br>
> >  <p><br>
> > -Patches should be sent to the mesa-dev mailing list for review:<br>
> > +Patches may be submitted to the Mesa project by<br>
> > +<a href="#mailing">email</a> or with a<br>
> > +GitLab <a href="#merge-request">merge request</a>. To prevent<br>
> > +duplicate code review, only use one method to submit your changes.<br>
> > +</p><br>
> ><br>
> <br>
> Do we want to require a cover-letter to be sent to the ML?  Ideally, we'd<br>
> just have a bot that makes one every time someone submits a MR and sends it<br>
> to the list.  Maybe someone just needs to write that bot.<br>
> <br>
> > +<br>
> > +<h3 id="mailing">Mailing Patches</h3><br>
> > +<br>
> > +<p><br>
> > +Patches may be sent to the mesa-dev mailing list for review:<br>
> >  <a href="<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a>"><br>
> >  <a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a></a>.<br>
> >  When submitting a patch make sure to use<br>
> > @@ -217,8 +228,63 @@ disabled before sending your patches. (Note that you<br>
> > may need to contact<br>
> >  your email administrator for this.)<br>
> >  </p><br>
> ><br>
> > +<h3 id="merge-request">GitLab Merge Requests</h3><br>
> > +<br>
> > +<p><br>
> > +  <a href="<a href="https://gitlab.freedesktop.org/mesa/mesa" rel="noreferrer" target="_blank">https://gitlab.freedesktop.org/mesa/mesa</a>">GitLab</a> Merge<br>
> > +  Requests (MR) can also be used to submit patches for Mesa.<br>
> > +</p><br>
> > +<br>
> > +<p><br>
> > +  If the MR may have interest for most of the Mesa community, you can<br>
> > +  send an email to the mesa-dev email list including a link to the MR.<br>
> > +  Don't send the patch to mesa-dev, just the MR link.<br>
<br>
Regarding the cover-letter, I put in this weasel worded sentence.<br>
Should it instead say this is required and that it should be a git<br>
format-patch generated cover letter?<br></blockquote><div><br></div><div>I didn't read far enough....  No, I don't think we need to require git-send-email formatted.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Or, should we drop it entirely and assume we'll get an automated way<br>
to send an email to the list whenever a new MR is opened?<br>
<br>
Relatedly, I think it might be possible to enable an irc channel to be<br>
notified about pushes and MR's. Not sure if it'd be a good idea, or<br>
maybe too noisy.<br></blockquote><div><br></div><div>We should totally have an IRC bot.  We had one for wayland and weston when I was working on those and it was great.  If it notifies us of every change, it may be too much but if it dumps something in the channel for every new MR, that shouldn't be bad at all.</div><div><br></div><div>Also,  regardless of the weasel-wording, this is</div><div><br></div><div>Acked-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br></div><div><br></div><div>--Jason<br></div></div></div>