<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Dec 11, 2019 at 12:02 PM Brian Paul <<a href="mailto:brianp@vmware.com">brianp@vmware.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">On 12/09/2019 04:07 PM, Dylan Baker wrote:<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>
<br>
I've had little time for Mesa work the past 18 months.  But I'd say:<br>
<br>
1. I don't think the mesa-dev list should be shut down nor purged from <br>
the documentation.<br>
<br>
2. I think the complete workflow for submitting patches via gitlab MRs <br>
should be documented on the Mesa site.  Being a gitlab newbie I wasted a <br>
lot of time a few months ago trying to figure out the details.  Mesa's <br>
"Submitting Patches" page is kind of a mess.  It would be great if <br>
someone could work on that.<br></blockquote><div><br></div>Yeah, the "Submitting Patches" page is useless, because it doesn't say the things that you need to know, and it says the things that you mostly do NOT need to know.</div><div class="gmail_quote"><br></div><div class="gmail_quote"> The steps for creating a MR are:</div><div class="gmail_quote">1*) Click "fork" in the Meso repo to create a private repo where you'll push branches for MRs.</div><div class="gmail_quote">2) git push your branch into your private repo</div><div class="gmail_quote">3) The "git push" command printed a link to create a MR for that branch. Open it.</div><div class="gmail_quote">4*) Add labels, then click "Allow commits from members who can merge to the target branch" and click Submit.</div><div class="gmail_quote"><br></div><div class="gmail_quote">Merging a branch:</div><div class="gmail_quote">1*) Check that all CI pipelines have succeeded.<br></div><div class="gmail_quote">2) Rebase your local branch and force-push to your private branch.</div><div class="gmail_quote">3) Wait ~10 seconds.</div><div class="gmail_quote">4) Push that branch to master (git push origin HEAD:master). Gitlab will automatically mark the MR as "Merged".</div><div class="gmail_quote"><br></div><div class="gmail_quote">* these are the only steps when you need to use the web UI.</div><div class="gmail_quote"><br></div><div class="gmail_quote">Marek<br></div></div>