Ok; one more request. :)<br><br>Can the .ver-svn temporary file go in the build products directory, not the source dir?<br><br> - Daniel<br><br><div class="gmail_quote">On Wed, Mar 18, 2009 at 12:05 PM, Daniel Dunbar <span dir="ltr"><<a href="mailto:daniel@zuster.org">daniel@zuster.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Nice! Two minor requests:<br><br>1. Can tools/driver/driver.cpp pass this in instead of having inside the lib. The version should be particular to the client not the library.<br>
<br>2. Not sure about this; but do you have enough Makefile foo that we could automatically do this every time the clang-driver executable was linked (but only then)?<br><font color="#888888">
<br> - Daniel</font><div><div></div><div class="h5"><br><br><div class="gmail_quote">On Wed, Mar 18, 2009 at 11:51 AM, Mike Stump <span dir="ltr"><<a href="mailto:mrs@apple.com" target="_blank">mrs@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div>On Mar 18, 2009, at 10:39 AM, Daniel Dunbar wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
ccc version 1.0 (<a href="http://llvm.org/svn/llvm-project/cfe/trunk" target="_blank">http://llvm.org/svn/llvm-project/cfe/trunk</a>)<br>
<br>
which I think is fine.  If someone tags clang, say for an llvm release, it will have the release tag.  If we want to add the svn revision information, a la gcc, we could do that, but it is a harder to get at and isn't always available (svn export say).<br>


<br>
Yes, this is what I meant.<br>
</blockquote>
<br></div>
Ah, fixed:<br>
<br>
$ clang-driver  -v<br>
ccc version 1.0 (<a href="https://mrs" target="_blank">https://mrs</a>@<a href="http://llvm.org/svn/llvm-project/cfe/trunk" target="_blank">llvm.org/svn/llvm-project/cfe/trunk</a> 67214:67220)<br>
$ cvn ci<br>
$ make<br>
$ clang-driver  -v<br>
ccc version 1.0 (<a href="https://mrs" target="_blank">https://mrs</a>@<a href="http://llvm.org/svn/llvm-project/cfe/trunk" target="_blank">llvm.org/svn/llvm-project/cfe/trunk</a> 67220)<br>
<br>
:-)<br>
</blockquote></div><br>
</div></div></blockquote></div><br>