Skip to content

Commit

Permalink
Add back the top-level target for 'llvm-config' and the dependency edge
Browse files Browse the repository at this point in the history
to force it to build after all library targets so it has complete
dependency information. This should fix broken 'make install' with
CMake.

This is a partial revert of r143540, but it doesn't revert the most
important part of that change: removing the dependency edge from LLVM
tools to the llvm-config script.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@143548 91177308-0d34-0410-b5e6-96231b3b80d8
  • Loading branch information
chandlerc committed Nov 2, 2011
1 parent 8763302 commit 05a484b
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions tools/llvm-config/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -125,6 +125,13 @@ add_custom_command(OUTPUT ${LLVM_CONFIG}
COMMENT "Building llvm-config script."
)

add_custom_target(llvm-config.target ALL
DEPENDS ${LLVM_CONFIG})

# Ensure we build llvm-config after we build all of the libraries so that we
# have their full dependencies.
add_dependencies(llvm-config.target ${llvm_libs})

install(FILES ${LLVM_CONFIG}
PERMISSIONS OWNER_READ OWNER_WRITE OWNER_EXECUTE GROUP_READ GROUP_EXECUTE
WORLD_READ WORLD_EXECUTE
Expand Down

0 comments on commit 05a484b

Please sign in to comment.