Opened 10 years ago

Closed 10 years ago

#45168 closed defect (fixed)

chromaprint: fails to install libchromaprint.pc

Reported by: dbevans (David B. Evans) Owned by: kurthindenburg (Kurt Hindenburg)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: cooljeanius (Eric Gallager)
Port: chromaprint

Description

When building on Apple platforms, port chromaprint intentionally fails to install libchromaprint.pc. See CMakeLists.txt:

if(NOT APPLE AND NOT BUILD_FRAMEWORK)
        configure_file(${CMAKE_CURRENT_SOURCE_DIR}/libchromaprint.pc.cmake ${CMAKE_CURRENT_BINARY_DIR}/libchromaprint.pc)
        install(FILES ${CMAKE_CURRENT_BINARY_DIR}/libchromaprint.pc DESTINATION ${LIB_INSTALL_DIR}/pkgconfig)
endif()

This is inappropriate for MacPorts and causes some other ports (e.g. gstreamer1-gst-plugins-bad) to fail to detect libchromaprint during the configure phase even if it is installed.

Change History (4)

comment:1 Changed 10 years ago by cooljeanius (Eric Gallager)

Sounds like an upstream issue, although I suppose it does make sense to just patch it in the meantime while waiting on upstream...

comment:2 Changed 10 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

comment:3 Changed 10 years ago by dbevans (David B. Evans)

The upstream assumption is that if it an Apple (native) platform or a framework is being built then the pkg-config file is not needed because it is a Linux-ism and not required for an Apple native environment or a framework. An equivalent logical expression is NOT (APPLE OR BUILD_FRAMEWORK).

However, in MacPorts we are often using Linux-style builds hence the need for the pkg-config file when the non-framework library is being installed. So I think using NOT (APPLE AND BUILD_FRAMEWORK) or just NOT BUILD_FRAMEWORK would be more appropriate for MacPorts.

A reasonable resolution IMO would be to apply a patch now, since this breaks pkg-config based configurations and open an issue upstream but I'm content to leave it to the maintainer's judgement.

Last edited 10 years ago by dbevans (David B. Evans) (previous) (diff)

comment:4 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.