Skip to content
This repository was archived by the owner on Aug 30, 2022. It is now read-only.

cmake: Use GNUInstallDirs instead of hardcoding lib destination #102

Merged
merged 1 commit into from
May 16, 2018
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
35 changes: 20 additions & 15 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -340,12 +340,17 @@ endif()

# Installation (https://github.com/forexample/package-example)

# Introduce variables:
# * CMAKE_INSTALL_LIBDIR
# * CMAKE_INSTALL_BINDIR
# * CMAKE_INSTALL_INCLUDEDIR
include(GNUInstallDirs)

# Layout. This works for all platforms:
# * <prefix>/lib/cmake/<PROJECT-NAME>
# * <prefix>/lib/
# * <prefix>/lib*/cmake/<PROJECT-NAME>
# * <prefix>/lib*/
# * <prefix>/include/
set(config_install_dir "lib/cmake/${PROJECT_NAME}")
set(include_install_dir "include")
set(config_install_dir "${CMAKE_INSTALL_LIBDIR}/cmake/${PROJECT_NAME}")

set(generated_dir "${CMAKE_CURRENT_BINARY_DIR}/generated")

Expand Down Expand Up @@ -376,42 +381,42 @@ configure_package_config_file(
)

# Targets:
# * <prefix>/lib/libjaegertracing.a
# * <prefix>/lib/libjaegertracing.so
# * <prefix>/lib*/libjaegertracing.a
# * <prefix>/lib*/libjaegertracing.so
# * header location after install: <prefix>/include/jaegertracing/Tracer.h
# * headers can be included by C++ code `#include <jaegertracing/Tracer.h>`
install(
TARGETS ${JAEGERTRACING_LIBS}
EXPORT "${TARGETS_EXPORT_NAME}"
LIBRARY DESTINATION "lib"
ARCHIVE DESTINATION "lib"
RUNTIME DESTINATION "bin"
INCLUDES DESTINATION "${include_install_dir}"
LIBRARY DESTINATION "${CMAKE_INSTALL_LIBDIR}"
ARCHIVE DESTINATION "${CMAKE_INSTALL_LIBDIR}"
RUNTIME DESTINATION "${CMAKE_INSTALL_BINDIR}"
INCLUDES DESTINATION "${CMAKE_INSTALL_INCLUDEDIR}"
)

# Headers:
# * src/jaegertracing/Tracer.h -> <prefix>/include/jaegertracing/Tracer.h
install(DIRECTORY "src/jaegertracing"
DESTINATION "${include_install_dir}"
DESTINATION "${CMAKE_INSTALL_INCLUDEDIR}"
FILES_MATCHING
PATTERN "*.h"
PATTERN "testutils/*.h" EXCLUDE)

# * build/src/jaegertracing/Constants.h ->
# <prefix>/include/jaegertracing/Constants.h
install(FILES "${CMAKE_CURRENT_BINARY_DIR}/src/jaegertracing/Constants.h"
DESTINATION "${include_install_dir}/jaegertracing")
DESTINATION "${CMAKE_INSTALL_INCLUDEDIR}/jaegertracing")

# Config
# * <prefix>/lib/cmake/jaegertracing/jaegertracingConfig.cmake
# * <prefix>/lib/cmake/jaegertracing/jaegertracingConfigVersion.cmake
# * <prefix>/lib*/cmake/jaegertracing/jaegertracingConfig.cmake
# * <prefix>/lib*/cmake/jaegertracing/jaegertracingConfigVersion.cmake
install(
FILES "${project_config}" "${version_config}"
DESTINATION "${config_install_dir}"
)

# Config
# * <prefix>/lib/cmake/jaegertracing/jaegertracingTargets.cmake
# * <prefix>/lib*/cmake/jaegertracing/jaegertracingTargets.cmake
install(
EXPORT "${TARGETS_EXPORT_NAME}"
NAMESPACE "${namespace}"
Expand Down