On Friday, 27 May 2016, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">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 href="javascript:_e(%7B%7D,'cvml','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">From: Emil Velikov <<a href="javascript:_e(%7B%7D,'cvml','emil.velikov@collabora.com');" target="_blank">emil.velikov@collabora.com</a>><br>
<br>
Otherwise we'll error out if build from a release tarball.<br>
<br>
Cc: Jason Ekstrand <<a href="javascript:_e(%7B%7D,'cvml','jason.ekstrand@intel.com');" target="_blank">jason.ekstrand@intel.com</a>><br>
Cc: Kristian Høgsberg Kristensen <<a href="javascript:_e(%7B%7D,'cvml','krh@bitplanet.net');" target="_blank">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><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><div> </div><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 href="javascript:_e(%7B%7D,'cvml','jason@jlekstrand.net');" target="_blank">jason@jlekstrand.net</a>></div></div></div></div></blockquote><div><br></div><div>Thanks.</div><div>Emil</div>