<div dir="ltr"><div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Mon, Dec 17, 2018 at 2:13 PM Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</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"><div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Mon, Dec 17, 2018 at 1:53 PM Eric Anholt <<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</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">Jason Ekstrand <<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>> writes:<br>
<br>
> I don't know if it was actually in the doc that Jordan wrote up but it's<br>
> courteous of you to send a quick e-mail to the mailing list when you create<br>
> a new MR so that people who aren't regularly trolling the list of MRs are<br>
> at least aware that it exists.  Of the 20 MRs that have been posted so far,<br>
> I think I'm the only one doing this.  I'm a big fan of MRs but I also don't<br>
> want us MR fans to anger the list. :-)<br>
<br>
The conclusion of the MR discussion was that notifying the list was<br>
optional, I thought.<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>
+</p><br></blockquote><div><br></div><div>Yeah, I think the problem is that "interest for most of the Mesa community" is very vague.  What it really should mean is "unless you're very sure that everyone who cares is skimming through MRs".  I've seen multiple MRs which touch on st/mesa stuff in reasonably generic ways and I'm pretty sure the radeon devs and other gallium types were some of the less excited about MRs.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
The MR process is heavier-weight than I hoped for, given that we're<br>
currently requiring rebasing tags into commit messages.  I don't want to<br>
add more overhead to it if we don't have to.<br></blockquote><div><br></div><div>I agree.  I think the real solution here is that we just need a bot.</div></div></div></blockquote><div><br></div><div>I took a swing at this.  This python script will do the trick if we can find somewhere to host it and harden it a bit against being a spam-magnet:</div><div><br></div><div><a href="https://paste.fedoraproject.org/paste/EGGpnxNNWEmadt0fWJEfGg">https://paste.fedoraproject.org/paste/EGGpnxNNWEmadt0fWJEfGg</a></div><div><br></div><div>It's really not all that hard, it's just a bit annoying that GitLab requires you to run server-side scripting in a web server in order to implement a simple hook.</div><div><br></div><div>Unfortunately, Ilia's request for a diffstat is easier said than done.  In order to do that, the script would have to actually pull the repo and run git commands.  It could be done but would be a *lot* more work.  I'm not going to attempt that today nor do I really want to get stuck maintaining a python git hook library. :-P</div><div><br></div><div>--Jason<br></div></div></div></div>