Opened 3 years ago

Last modified 9 months ago

#54131 assigned defect

llvm-3.7 @3.7.1_4: Faulty LLVMConfig.cmake

Reported by: csinz (Carsten Sinz) Owned by: jeremyhu (Jeremy Huddleston Sequoia)
Priority: Normal Milestone:
Component: ports Version: 2.4.1
Keywords: Cc: larryv (Lawrence Velázquez), kencu (Ken)
Port: llvm-3.7

Description

The file share/llvm/cmake/LLVMConfig.cmake is broken. Instead of the lines 

# Compute the CMake directory from the LLVMConfig.cmake file location.
get_filename_component(_LLVM_CMAKE_DIR "${CMAKE_CURRENT_LIST_FILE}" PATH)
# Compute the installation prefix from the LLVMConfig.cmake file location.
get_filename_component(LLVM_INSTALL_PREFIX "${CMAKE_CURRENT_LIST_FILE}" PATH) 
get_filename_component(LLVM_INSTALL_PREFIX "${LLVM_INSTALL_PREFIX}" PATH)
get_filename_component(LLVM_INSTALL_PREFIX "${LLVM_INSTALL_PREFIX}" PATH)
get_filename_component(LLVM_INSTALL_PREFIX "${LLVM_INSTALL_PREFIX}" PATH)
set(_LLVM_LIBRARY_DIR "${LLVM_INSTALL_PREFIX}/lib")

there is a long line, in which newlines are erroneously replaced by 'n' resulting in 

# Compute the CMake directory from the LLVMConfig.cmake file location.n get_filename_component(_LLVM_CMAKE_DIR "${CMAKE_CURRENT_LIST_FILE}" PATH)n # Compute the installation prefix from the LLVMConfig.cmake file location.n get_filename_component(LLVM_INSTALL_PREFIX "${CMAKE_CURRENT_LIST_FILE}" PATH)n get_filename_component(LLVM_INSTALL_PREFIX "${LLVM_INSTALL_PREFIX}" PATH)n get_filename_component(LLVM_INSTALL_PREFIX "${LLVM_INSTALL_PREFIX}" PATH)n get_filename_component(LLVM_INSTALL_PREFIX "${LLVM_INSTALL_PREFIX}" PATH)n set(_LLVM_LIBRARY_DIR "${LLVM_INSTALL_PREFIX}/lib")

The error occurs in the binary llvm-3.7-3.7.1_4.darwin_15.x86_64.tbz2 (or perhaps llvm-3.7-3.7.1_4.darwin_15.x86_64.tbz2.rmd160).

Change History (3)

comment:1 Changed 3 years ago by mf2k (Frank Schima)

Cc: larryv added
Owner: set to jeremyhu
Status: new → assigned

In the future, please Cc the port maintainers (port info --maintainers llvm-3.7), if any.

comment:2 Changed 3 years ago by csinz (Carsten Sinz)

Sorry, I thought, I had added them.

comment:3 Changed 9 months ago by kencu (Ken)

Cc: kencu added
Note: See TracTickets for help on using tickets.