This will remove the ability to use llvm-lit script even if source tree is available.
Can you put this behind a check that for standalone build? Also, why was this just found? shouldnt this have been failing on Apple’s libcxx for the past several months? On Wed, Nov 1, 2017 at 4:38 PM Volodymyr Sapsai via Phabricator < [email protected]> wrote: > vsapsai created this revision. > Herald added a subscriber: mgorny. > > Fixes error > > > CMake Error at test/CMakeLists.txt:52 (configure_lit_site_cfg): > > Unknown CMake command "configure_lit_site_cfg". > > configure_lit_site_cfg is defined in AddLLVM module and not available > without LLVM source tree. Revert back to configure_file (reverts part of > r313643). It is possible as libcxx/test/lit.site.cfg.in doesn't use > latest lit capabilities. > > Tested with > > - in-tree libcxx - no change in generated lit.site.cfg and running tests; > - standalone libcxx with lit checked out next to it - no CMake errors. > > If there are other tricky configurations worth testing, let me know. > > rdar://problem/35303262 > > > https://reviews.llvm.org/D39520 > > Files: > libcxx/test/CMakeLists.txt > > > Index: libcxx/test/CMakeLists.txt > =================================================================== > --- libcxx/test/CMakeLists.txt > +++ libcxx/test/CMakeLists.txt > @@ -49,9 +49,10 @@ > > set(AUTO_GEN_COMMENT "## Autogenerated by libcxx configuration.\n# Do not > edit!") > > -configure_lit_site_cfg( > +configure_file( > ${CMAKE_CURRENT_SOURCE_DIR}/lit.site.cfg.in > - ${CMAKE_CURRENT_BINARY_DIR}/lit.site.cfg) > + ${CMAKE_CURRENT_BINARY_DIR}/lit.site.cfg > + @ONLY) > > set(LIBCXX_TEST_DEPS "") > > > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
