<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 20, 2015 at 5:05 PM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class=""><div class="h5">On 21 November 2015 at 00:31, Nanley Chery <<a href="mailto:nanleychery@gmail.com">nanleychery@gmail.com</a>> wrote:<br>
> On Fri, Nov 20, 2015 at 3:38 PM, Chad Versace <<a href="mailto:chad.versace@intel.com">chad.versace@intel.com</a>><br>
> wrote:<br>
>><br>
>> On Fri 20 Nov 2015, Nanley Chery wrote:<br>
>> > On Fri, Nov 20, 2015 at 8:49 AM, Brian Paul <<a href="mailto:brian.e.paul@gmail.com">brian.e.paul@gmail.com</a>><br>
>> > wrote:<br>
>> ><br>
>> > > On Thu, Nov 19, 2015 at 7:04 PM, Ian Romanick <<a href="mailto:idr@freedesktop.org">idr@freedesktop.org</a>><br>
>> > > wrote:<br>
>><br>
>> > >> I think #2 might be better, but there's a lot of churn.  I don't know<br>
>> > >> that we want that much churn right around the time of the release<br>
>> > >> branch<br>
>> > >> point, and I think it would be good to have this resolved in 11.1.  I<br>
>> > >> also have a few bits of feedback in #2, so it might take a couple<br>
>> > >> iterations before that could land.<br>
>> > >><br>
>> > >><br>
>> > > Since 11.1 is coming up, can we go with the simpler #1 for now, then<br>
>> > > go to<br>
>> > > #2 after 11.1 branches?<br>
>> > ><br>
>> > ><br>
>> > I don't mind going this route. Unfortunately, I'm running into presently<br>
>> > unexplainable linker errors in the process of making the gtest for this<br>
>> > feature.<br>
>><br>
>> Nanley, you're branch has two patches. If you remove the gtest patch<br>
>> (patch #2) from your branch, does everything work as expected? If so,<br>
>> then I think it's best to get patch #1 into<br>
>> 11.1 before the Emil creates the branchpoint, and worry about the gtest<br>
>> afterwards.<br>
><br>
><br>
> I've been told that most people wait 24 hours before pushing a patch that's<br>
> been given an Rb. Would it be acceptable to push it immediately given the<br>
> circumstances?<br>
><br>
</div></div>Get it out and reviewed please. This is a bugfix I'll pick it once<br>
it's ready, although add the mesa-stable line in there just in case<br>
:-)<br>
<br></blockquote><div><br>It recently has been reviewed:<br><a href="http://lists.freedesktop.org/archives/mesa-dev/2015-November/101027.html">http://lists.freedesktop.org/archives/mesa-dev/2015-November/101027.html</a><br><br></div><div>I was just wondering if I could push it to master now.<br><br></div><div>Thanks for the reply,<br></div><div>Nanley<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">
Thanks<br>
<span class=""><font color="#888888">Emil<br>
</font></span></blockquote></div><br></div></div>