<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Dec 10, 2019 at 4:59 PM Dylan Baker <<a href="mailto:dylan@pnwbakers.com">dylan@pnwbakers.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Quoting Zebediah Figura (2019-12-10 10:58:45)<br>
> On 12/10/19 12:21 PM, Matt Turner wrote:<br>
> > On Mon, Dec 9, 2019 at 6:07 PM Dylan Baker <<a href="mailto:dylan@pnwbakers.com" target="_blank">dylan@pnwbakers.com</a>> wrote:<br>
> >><br>
> >> Hi everyone,<br>
> >><br>
> >> I think its time we discussed whether we're going to continue to do patch review<br>
> >> on the mailing list, or if it it should all go through gitlab. I think we should<br>
> >> stop using the mailing list, here are some reasons:<br>
> >><br>
> >> 1) Most development is happening on gitlab at this point, patches on the mailing<br>
> >>     list are often overlooked<br>
> >> 2) The mailing list bypasses CI which potentially breaks the build<br>
> >> 3) Probably more reasons I'm forgetting.<br>
> > <br>
> > I think effectively we're already there.<br>
> > <br>
> > What concrete change would you propose?<br>
> <br>
> Removing mention of the mailing list from documentation would be nice. <br>
> Also, currently the README implies that the mailing list is not only <br>
> acceptable but preferred: "Note that Mesa uses email mailing-lists for <br>
> patches submission, review and discussions."<br>
<br>
This would be a good start, I don't know what else to do to stop people from<br>
sending patches, maybe mailman has some kind of hook we can add to spot patches<br>
and mention that gitlab is preferred? No idea.<br></blockquote><div><br></div><div>I think updating README type docs is sufficient and I think it's probably time we did that.  If someone sends a patch to the ML anyway, we should just go ahead and review it and suggest that next time they open an MR next time.  I don't think we need a hook or to do anything more forceful.  If the odd patch ends up on the mailing list, it won't hurt anyone.</div><div><br></div><div>As far about "OMG what if a drive-by person sends a patch!" goes.... I don't think that's all that likely.  New contributors are far more likely to understand the PR model from experience with GitHub than they are to know how to send patches.  (Unless they're already a long-time contributor to other systems level Linux projects but new to Mesa).  If it does happen, we shouldn't make a big stink about it.  We should just review the patch and suggest a pull request for next time.  If the patch needs more than just rubber-stamp review, maybe say "I've got some comments, mind making a PR so we can discuss it there?"<br></div><div><br></div><div>If people are all that concerned about CI, we can say that if you are pushing patches on behalf of someone else, you should create a MR and merge it that way.</div><div><br></div><div>--Jason<br></div></div></div>