Opened 8 years ago

Closed 3 years ago

Last modified 3 years ago

#54131 closed defect (wontfix)

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), cooljeanius (Eric Gallager)
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 (8)

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

Cc: larryv added
Owner: set to jeremyhu
Status: newassigned

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

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

Sorry, I thought, I had added them.

comment:3 Changed 5 years ago by kencu (Ken)

Cc: kencu added

comment:4 Changed 3 years ago by cooljeanius (Eric Gallager)

Cc: cooljeanius added

comment:5 Changed 3 years ago by kencu (Ken)

Cc: kencu removed

comment:6 Changed 3 years ago by kencu (Ken)

Resolution: wontfix
Status: assignedclosed

MacPorts does not use cmake to build clang 3.7

comment:7 Changed 3 years ago by jmroot (Joshua Root)

And just to be clear, there are no other clients of llvm-3.7.

comment:8 Changed 3 years ago by kencu (Ken)

Oh, yes, you are right Josh. I was cruising through old tickets that looked ripe for closing, and I misread this as a build with cmake issue. It is not actually that.

Note: See TracTickets for help on using tickets.