<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 20, 2015 at 3:38 PM, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@intel.com" target="_blank">chad.versace@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>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" target="_blank">brian.e.paul@gmail.com</a>> wrote:<br>
><br>
> > On Thu, Nov 19, 2015 at 7:04 PM, Ian Romanick <<a href="mailto:idr@freedesktop.org" target="_blank">idr@freedesktop.org</a>> wrote:<br>
<br>
</span><span>> >> 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 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 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>
</span>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>
</blockquote></div><br></div><div class="gmail_extra">I've been told that most people wait 24 hours before pushing a patch that's been given an Rb. Would it be acceptable to push it immediately given the circumstances?<br><br></div><div class="gmail_extra">Regards,<br></div><div class="gmail_extra">Nanley<br></div><div class="gmail_extra"><br></div></div>