vsk added inline comments.
================ Comment at: lldb/cmake/modules/LLDBFramework.cmake:142-147 + add_custom_command(TARGET liblldb POST_BUILD + COMMAND ${CMAKE_COMMAND} -E create_symlink + Versions/Current/XPCServices + ${LLDB_FRAMEWORK_ABSOLUTE_BUILD_DIR}/LLDB.framework/XPCServices + COMMENT "LLDB.framework: create symlink XPCServices" + ) ---------------- smeenai wrote: > This bit appears to have been introduced by this revision, without any > equivalent existing before. This is causing issues for us when we build an > LLDB framework in LLVM 12, because `Versions/Current/XPCServices` doesn't > exist, so the created symlink is dangling. Is there some other target which > should be creating that directory? (This is the only instance of > `XPCServices` I see in LLDB's build.) Thanks for catching this. I'm curious how you spotted this. It looks like we do have a target that sets up this directory, but that it's not upstream. It should be safe to remove this; we can reapply the change downstream if that happens. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85770/new/ https://reviews.llvm.org/D85770 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits