Give r159594 a whirl, and let me know how that looks?
On Mon, Jul 2, 2012 at 2:09 PM, Jordan Rose <[email protected]> wrote: > > On Jul 2, 2012, at 14:06 , Chandler Carruth <[email protected]> wrote: > > On Mon, Jul 2, 2012 at 2:03 PM, Jordan Rose <[email protected]> wrote: > >> >> On Jul 2, 2012, at 13:53 , Chandler Carruth <[email protected]> wrote: >> >> > 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. >> >> 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. > > > Ah, ok, I have a patch for that to. It's also pretty obvious/easy when you > describe it. Give me a moment. =] > > 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. > > > …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.) > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
