Hmm, I see why. Not sure how to fix.<div><br></div><div>The way we use lit is really annoying in this situation.</div><div><br></div><div>There are two ways to tell lit about a test suite:</div><div>1) You can pass it a directory at the root of the test suite on the commandline.</div>
<div>2) You can pass it a parent directory of the test suite and it will recurse through them</div><div><br></div><div>There are then three ways to tell it about a site config file for the test suite:</div><div>1) It can find one in the root of the directory passed on the commandline.</div>
<div>2) It can find one in a subdirectory that overrides the root one.</div><div>3) It can find one via a user parameter and manually override the site config in the non-site config.</div><div><br></div><div>We currently seem to be using *all three* of the latter systems, and both of the former systems. All at the same time.</div>
<div><br></div><div>The redundancy means that 'fixing' this at any one level doesn't actually fix it... I don't even know how to fix it at all levels. =/</div><div><br></div><div>I would really like to remove #2 from both of these. We already have a *local* config override that happens in subdirectories, being able to just arbitrarily nest testsuites seems to cause more harm than good. =/ It would have the side-effect of being the last thing required to fix the warning message you see.</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 2, 2012 at 2:44 PM, Jordan Rose <span dir="ltr"><<a href="mailto:jordan_rose@apple.com" target="_blank">jordan_rose@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div>…nope, still see it. It's just a warning, though, so it's not a priority.</div>
<div><div class="h5"><div><br></div><br><div><div>On Jul 2, 2012, at 14:37 , Chandler Carruth <<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>> wrote:</div><br><blockquote type="cite">Give r159594 a whirl, and let me know how that looks?<div class="gmail_extra">
<br><br><div class="gmail_quote">On Mon, Jul 2, 2012 at 2:09 PM, Jordan Rose <span dir="ltr"><<a href="mailto:jordan_rose@apple.com" target="_blank">jordan_rose@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div><br><div><div>On Jul 2, 2012, at 14:06 , Chandler Carruth <<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>> wrote:</div>

<br><blockquote type="cite"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jul 2, 2012 at 2:03 PM, Jordan Rose <span dir="ltr"><<a href="mailto:jordan_rose@apple.com" target="_blank">jordan_rose@apple.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><br>
On Jul 2, 2012, at 13:53 , Chandler Carruth <<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>> wrote:<br>
<br>
> It was obvious: r159582. I can't promise that's *enough* but it seems definitely a good initial step. Let me know if that helps.<br>
<br>
</div>Yes, it does! The only thing left is "Test suite 'Clang-Unit" contained no tests". And indeed, ClangUnitTests is not a target in my standalone-build project, but it and its subtargets do show up in my combined project.</blockquote>


<div><br></div><div>Ah, ok, I have a patch for that to. It's also pretty obvious/easy when you describe it. Give me a moment. =] </div><div><br></div><div>That said, the patch will remove the message. We can't build the Clang unittests in a standalone build. =/ At least, I'm not particularly interested in adding the various build steps necessary for that -- i'd rather make the not-standalone build just as fast for IDE users.</div>


</div></div>
</blockquote></div><br></div></div><div>…unfortunate, but fair enough. (I see that the add_unittest function does enough that it's not something we want duplicated in Clang's CMakeLists.txt.)</div></div><br>_______________________________________________<br>


cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br></div></div></div><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>