<div dir="ltr">On 23 April 2013 21:22, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@linux.intel.com" target="_blank">chad.versace@linux.intel.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div>On 04/23/2013 09:19 PM, Eric Anholt wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Chad Versace <<a href="mailto:chad.versace@linux.intel.com" target="_blank">chad.versace@linux.intel.com</a>> writes:<br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
On 04/23/2013 06:19 AM, Ian Romanick wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
On 04/23/2013 03:28 AM, Chad Versace wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
This allows maintainers/packagers/testers to tag the build with<br>
information that will be reported by GL_VERSION.<br>
<br>
If the environemt variable or make variable MESA_VERSION_STRING_EXTRA is<br>
</blockquote>
           environment<br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
set, then its values will appear in the GL_VERSION string immediately<br>
after "Mesa X.Y" and before "(git-xxxxxxx)".<br>
<br>
This patch implements supports MESA_VERSION_STRING_EXTRA only for Android.<br>
Other build systems are left as an excercise.<br>
</blockquote>
<br>
Why is this useful?<br>
</blockquote>
<br>
The commit message says why: it allows packagers to tag the build with info<br>
that gets reported by GL_VERSION.<br>
<br>
How is it used now? The Mesa that gets shipped bi-weekly internally in Intel<br>
for Android is not a snapshot of master nor of any stable branch. Yet, that<br>
snapshot of Mesa is "official", as far as Intel's Android efforts are concerned.<br>
The Android team is using this patch to inject into GL_VERSION the name of the bi-weekly<br>
Android release.<br>
<br>
Without this patch, you may one day get a bug report that says: "This bug exists<br>
in Mesa 9.2-devel (git-xxxxxxx)". Then, you will search for the git sha1, and<br>
unable to find it, yell bloody murder and close the bug as invalid. With this<br>
patch, though, you may get a bug report that says: "This bug exists in<br>
Mesa 9.2-devel otc-android-2013-04-23-blah (git-xxxxxxx)", and then you<br>
will know how to find that Mesa.<br>
</blockquote>
<br>
So along with a useless sha1, I have a useless date that doesn't tell me<br>
what Mesa it was actually based off of.  I say this having tried to<br>
figure out what Mesa actually got used in one of these android tests and<br>
failed.<br>
</blockquote>
<br></div></div>
Rather than jabs and parrying, let's turn this conversation into a<br>
constructive one. If you receive a<br>
communication about Mesa on Android, how do you want the Mesa version<br>
communicated to you so that you can fetch and then inspect that same Mesa?<br>
<br>
Keep in mind that the Android Mesa tree is regularly rebased onto master, so  the shas<br>
are not as useful as you would hope. Also know that we have git tags for<br>
the "useless" dates if you know where to fetch from.</blockquote><div><br></div><div>Would it be helpful to use "git describe --long" to generate the git portion of the GL_version string?  That's essentially what the Linux kernel build uses it for this kind of purpose.  It generates a  generates a string like this: "mesa-9.1-9-ga5c79b7", where "mesa-9.1" is the most recent annotated tag accessible from HEAD, "9" is the number of commits between that tag and HEAD, and "a5c79b7" is the SHA of HEAD.<br>
<br></div><div>If we do this we might want to start making annotated tags of master every now and again--currently the most recent annotated tag accessible from master is a tag called "snb-magic" from November 2010.<br>
</div></div></div></div>