<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Feb 4, 2019 at 6:25 PM Alyssa Rosenzweig <<a href="mailto:alyssa@rosenzweig.io">alyssa@rosenzweig.io</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">> 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></blockquote><div><br></div><div>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></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">
>  4. Don't commit code outside of Panfrost without review from someone who<br>
> has a decent claim to knowing what they're talking about in that area<br>
<br>
Acked-by: Alyssa Rosenzweig <<a href="mailto:alyssa@rosenzweig.io" target="_blank">alyssa@rosenzweig.io</a>><br>
<br>
>  5. Be nice and give stake-holders (who may not be in your time zone)<br>
> enough time to read their e-mail and review (at least 24 hours not<br>
> including week-ends) before pushing anything.<br>
<br>
Acked-by: Alyssa Rosenzweig <<a href="mailto:alyssa@rosenzweig.io" target="_blank">alyssa@rosenzweig.io</a>><br>
<br>
> Inside Panfrost, it's kind-of up to the Panfrost community developers what<br>
> you want to do about review.  If you've got enough people, I highly<br>
> recommend you do good code review.<br>
<br>
Downstream, we've been trying to enforce proper reivew. I review other<br>
people's code; I haggle people to review mine; you know how it is :)<br>
<br>
Thank you again!<br>
<br>
Alyssa<br>
</blockquote></div></div>