keith added inline comments.

================
Comment at: cmake/modules/LLDBFramework.cmake:41
+add_custom_target(lldb-framework-headers
+  DEPENDS ${framework_headers}
+  COMMAND ${LLDB_SOURCE_DIR}/scripts/framework-header-fix.sh
----------------
xiaobai wrote:
> This should not just depend on `framework_headers`, because those are in the 
> location `${CMAKE_CURRENT_BINARY_DIR}/FrameworkHeaders`. You want to run the 
> script on `$<TARGET_FILE_DIR:liblldb>/Headers`, which is not guaranteed to 
> exist or have been populated with headers when this actually gets run. See my 
> comment above.
Updated this by making it a POST_BUILD command instead


https://reviews.llvm.org/D49779



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to