2008/12/30 Talin <span dir="ltr"><<a href="mailto:viridia@gmail.com">viridia@gmail.com</a>></span><br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div>OK changes made and new patch attached.</div></blockquote><div><br>I'm not sure how I missed this the first time, or if it's new:<br><br>+++ unittests/TestMain.cpp    (revision 0)<br>+//===--- TestMail.cpp - unittest driver -----------------------------------===//<br>
<br>Same comment as previously. :-)<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div></div><div>On Tue, Dec 30, 2008 at 2:40 PM, Misha Brukman <span dir="ltr"><<a href="mailto:brukman@gmail.com" target="_blank">brukman@gmail.com</a>></span> wrote:<br>
</div><div class="gmail_quote"><div class="Ih2E3d"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="gmail_quote">2008/12/30 Misha Brukman <span dir="ltr"><<a href="mailto:brukman@gmail.com" target="_blank">brukman@gmail.com</a>></span><br>


<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">2008/12/30 Talin <span dir="ltr"><<a href="mailto:viridia@gmail.com" target="_blank">viridia@gmail.com</a>></span><br>
<div class="gmail_quote"><div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div>Some notes on the patch:</div><div><ul><li>This patch doesn't include googletest itself, that will need to be checked in separately. The source distribution will live in llvm/utils/unittest/googletest. (The reason for the extra directory level is so that the LLVM-specific makefiles can live in llvm/utils/unittest, while the googletest directory itself can be the pristine, unmodified distribution.)</li>



</ul></div></blockquote></div><div>Are you going to send a separate patch for this, or should I do this?</div></div></blockquote></div></blockquote><div><br></div></div><div>Please go ahead and do this. It seems silly (to me anyway) to convert the source tarball into a patch file.</div>
</div></blockquote><div><br>Yes, I figured this out after trying to see what's involved there.  I'm thinking we don't need to keep everything in the tarball, because it'll just bloat the LLVM distribution.  I think we should be fine with just the include/ and src/ directories, and I'll keep the COPYING file since it's the license, and add a README.LLVM file.  Sound good? <br>
</div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="gmail_quote"><div></div><div class="Ih2E3d">
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

<div class="gmail_quote"><div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

<div><ul><li></li>
<li>The individual tests are in llvm/unittests (note plural).</li><li>The makefile target to build and run the tests is "make unittest" (I wasn't sure if you could have a synthetic target that was the same as the name of a directory, so I made it "unittest" instead of "unittests")</li>



</ul></div></blockquote></div><div>I don't think that's an issue -- to recurse into a directory, you have to say "make -C dir [target]". </div></div></blockquote></div></blockquote></div></div></blockquote>
<div><br>So can we call the target to build and run unittests "unittests" or does it create problems?<br></div></div><br>