<div dir="ltr"><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Feb 4, 2019 at 6:43 PM Ilia Mirkin <<a href="mailto:imirkin@alum.mit.edu">imirkin@alum.mit.edu</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 Mon, Feb 4, 2019 at 7:38 PM Jason Ekstrand <<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>> wrote:<br>
><br>
> On Mon, Feb 4, 2019 at 6:25 PM Alyssa Rosenzweig <<a href="mailto:alyssa@rosenzweig.io" target="_blank">alyssa@rosenzweig.io</a>> wrote:<br>
>><br>
>> > Actually, I just gave you (Alyssa) push access... Also, as you'll (as<br>
>> > far as I understand) basically be owning the panfrost bits in mesa,<br>
>> > you should be able to commit to them.<br>
>><br>
>> Oh, thank you! :)<br>
>><br>
>> >  1. Don't break the build<br>
>><br>
>> Acked-by: Alyssa Rosenzweig <<a href="mailto:alyssa@rosenzweig.io" target="_blank">alyssa@rosenzweig.io</a>><br>
>><br>
>> >  2. No merge commits<br>
>><br>
>> Just to be clear, is the idea to just make sure everything applies<br>
>> cleanly / is a straightforward fast-forward, and if not, to rebase/squash<br>
>> so it does?<br>
><br>
><br>
> Roughtly?  I really did mean "no merge commits" which really just means linear history.  Ideally, you'd have something that roughly linearly works but that's a Panfrost quality thing. I'm sure there will be regressions all over the place as you work given that it's still a bit prototypey.<br>
<br>
An important point here is "bisectable". You shouldn't have commits<br>
like "fix this totally broken earlier commit in the series". Breakage<br>
can happen -- that's a fact of life -- but you should avoid having<br>
"known" breaking commits, since that will mess up bisects down the<br>
line.<br></blockquote><div><br></div><div>Right.  I just didn't want to sound too draconian because I know what the crazy prototype world looks like.  That said, the more linearly working and bisectable the better. <br></div></div></div>