<br><br><div class="gmail_quote"><div dir="ltr">On Thu, 22 Jun 2017 at 00:12 Matt Turner <<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Jun 21, 2017 at 3:50 PM, Mike Lothian <<a href="mailto:mike@fireburn.co.uk" target="_blank">mike@fireburn.co.uk</a>> wrote:<br>
> Do intel run mesa through any of their test boxes like they do with kernel<br>
> patches?<br>
<br>
Don't top quote.<br>
<br>
Yes, we have a CI system that we use extensively and has massively<br>
reduced the number of regressions we have. Immense thanks to Mark<br>
Janes for all of the work he's done on it.<br>
<br>
In this instance, the CI didn't catch it because it tests debug builds<br>
(as you'd expect), so the assert and the function it called were<br>
executed. In release builds, however, the assert and that necessary<br>
function call were removed.<br></blockquote></div><div><br></div><div>Sorry, top posting is the default in Inbox and there doesn't appear to be away to change it</div><div><br></div><div>I wasn't criticising, it's very rare that a serious bug creaps into master, having said that it would have been nice to have a response on IRC when the issue was pointed out </div><div><br></div><div>Least it's fixed now and I'm back to testing master again</div><div><br></div><div>Mike</div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
</blockquote></div>