On Tue, Apr 28, 2015 at 1:55 PM, Nico Weber <tha...@chromium.org> wrote:
> ================
> Comment at: cmake/LLDBDependencies.cmake:100
> @@ +99,3 @@
> +if ( CMAKE_SYSTEM_NAME MATCHES "Darwin" )
> + set(LLDB_VERS_GENERATED_FILE ${LLDB_BINARY_DIR}/source/LLDB_vers.c)
> + add_custom_command(OUTPUT ${LLDB_VERS_GENERATED_FILE}
> ----------------
> thakis wrote:
> > This doesn't look correct. Now every file that includes LLDBDependencies
> will have a custom command that builds tools/lldb/source/LLDB_vers.c. With
> ninja, this causes "ninja: warning: multiple rules generate
> tools/lldb/source/LLDB_vers.c. builds involving this target will not be
> correct; continuing anyway" for example (since there's more than one
> CMakeLists.txt that includes LLDBDependencies.txt).
> >
> > Maybe this rule could stay where it was?
> Ping? This is still broken 4 months later.
>
One last ping. If I don't hear back I'll try to fix it myself.
_______________________________________________
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
