On Mon, 2018-12-17 at 16:13 -0600, Jason Ekstrand wrote: > On Mon, Dec 17, 2018 at 2:13 PM Jason Ekstrand <[email protected]> > wrote: > > On Mon, Dec 17, 2018 at 1:53 PM Eric Anholt <[email protected]> > > wrote: > > > Jason Ekstrand <[email protected]> writes: > > > > > > > I don't know if it was actually in the doc that Jordan wrote up > > > but it's > > > > courteous of you to send a quick e-mail to the mailing list > > > when you create > > > > a new MR so that people who aren't regularly trolling the list > > > of MRs are > > > > at least aware that it exists. Of the 20 MRs that have been > > > posted so far, > > > > I think I'm the only one doing this. I'm a big fan of MRs but > > > I also don't > > > > want us MR fans to anger the list. :-) > > > > > > The conclusion of the MR discussion was that notifying the list > > > was > > > optional, I thought. > > > > > > +<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. > > > +</p> > > > > 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. > > > > > The MR process is heavier-weight than I hoped for, given that > > > we're > > > currently requiring rebasing tags into commit messages. I don't > > > want to > > > add more overhead to it if we don't have to. > > > > I agree. I think the real solution here is that we just need a > > bot. > > > > 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: > > https://paste.fedoraproject.org/paste/EGGpnxNNWEmadt0fWJEfGg > > 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. > > 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
Can't we get this from the gitlab API? https://docs.gitlab.com/ee/api/merge_requests.html#get-single-mr-changes _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
