<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, May 27, 2016 at 10:06 AM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Friday, 27 May 2016, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, May 27, 2016 at 7:35 AM, Emil Velikov <span dir="ltr"><<a>emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">From: Emil Velikov <<a>emil.velikov@collabora.com</a>><br>
<br>
Otherwise we'll error out if build from a release tarball.<br>
<br>
Cc: Jason Ekstrand <<a>jason.ekstrand@intel.com</a>><br>
Cc: Kristian Høgsberg Kristensen <<a>krh@bitplanet.net</a>><br>
---<br>
This patch isn't going to fly obviously but is just enough to get things<br>
building ;-)<br></blockquote><div><br></div><div>You're right.  It won't. :-)<br><br></div><div>I think what we need is to ship something that provides MESA_GIT_SHA1 in the tarball.  The purpose of this is to ensure that any shader caches get 100% invalidated if the mesa version changes at all.  When Timothy lands his shader cache stuff, we'll need this in the GL driver as well so we might as well find a solution now.<br><br></div></div></div></div></blockquote></span><div>Indeed. I was looking at how GL does it only to find it isn't wired up yet. And yes shipping git_sha1.h sounds like a good solution. Question is do we want to print the sha in other places (GL_VERSION iirc) for release builds ? I'd go with yes on that one as well.</div><div><br></div><div>I'm on the move(train) atm so it might be a little late in the day for anyone to review 'v2'.</div></blockquote><div><br></div><div>It's not late in my day.  It's not even noon yet. :-)<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>All of the other (not this one) anv/isl patches in the series are<br><br></div><div>Reviewed-by: Jason Ekstrand <<a>jason@jlekstrand.net</a>></div></div></div></div></blockquote><div><br></div></span><div>Thanks.</div><span class="HOEnZb"><font color="#888888"><div>Emil</div>
</font></span></blockquote></div><br></div></div>