Skip to content

Commit fb8bd0b

Browse files
authored
A bunch of CMake fixes (#101)
1 parent 2c7d355 commit fb8bd0b

File tree

3 files changed

+9
-78
lines changed

3 files changed

+9
-78
lines changed

CMakeLists.txt

Lines changed: 8 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -1,29 +1,14 @@
1-
cmake_minimum_required(VERSION 3.9)
1+
cmake_minimum_required(VERSION 3.9...3.31)
22

33
option(STATIC_CRT "Windows specific option that to specify static/dynamic run-time library" OFF)
44

55
project (aws-checksums C)
66

7-
if (DEFINED CMAKE_PREFIX_PATH)
8-
file(TO_CMAKE_PATH "${CMAKE_PREFIX_PATH}" CMAKE_PREFIX_PATH)
7+
if (NOT IN_SOURCE_BUILD)
8+
# this is required so we can use aws-c-common's CMake modules
9+
find_package(aws-c-common REQUIRED)
910
endif()
1011

11-
if (DEFINED CMAKE_INSTALL_PREFIX)
12-
file(TO_CMAKE_PATH "${CMAKE_INSTALL_PREFIX}" CMAKE_INSTALL_PREFIX)
13-
endif()
14-
15-
if (UNIX AND NOT APPLE)
16-
include(GNUInstallDirs)
17-
elseif(NOT DEFINED CMAKE_INSTALL_LIBDIR)
18-
set(CMAKE_INSTALL_LIBDIR "lib")
19-
endif()
20-
21-
# This is required in order to append /lib/cmake to each element in CMAKE_PREFIX_PATH
22-
set(AWS_MODULE_DIR "/${CMAKE_INSTALL_LIBDIR}/cmake")
23-
string(REPLACE ";" "${AWS_MODULE_DIR};" AWS_MODULE_PATH "${CMAKE_PREFIX_PATH}${AWS_MODULE_DIR}")
24-
# Append that generated list to the module search path
25-
list(APPEND CMAKE_MODULE_PATH ${AWS_MODULE_PATH})
26-
2712
include(AwsSIMD)
2813
include(AwsCFlags)
2914
include(AwsCheckHeaders)
@@ -32,6 +17,7 @@ include(AwsSanitizers)
3217
include(CheckCCompilerFlag)
3318
include(AwsFindPackage)
3419
include(AwsFeatureTests)
20+
include(GNUInstallDirs)
3521

3622
file(GLOB AWS_CHECKSUMS_HEADERS
3723
"include/aws/checksums/*.h"
@@ -137,7 +123,7 @@ aws_use_package(aws-c-common)
137123
target_link_libraries(${PROJECT_NAME} PUBLIC ${DEP_AWS_LIBS})
138124
aws_prepare_shared_lib_exports(${PROJECT_NAME})
139125

140-
install(FILES ${AWS_CHECKSUMS_HEADERS} DESTINATION "include/aws/checksums" COMPONENT Development)
126+
install(FILES ${AWS_CHECKSUMS_HEADERS} DESTINATION "${CMAKE_INSTALL_INCLUDEDIR}/aws/checksums" COMPONENT Development)
141127

142128
if (BUILD_SHARED_LIBS)
143129
set (TARGET_DIR "shared")
@@ -146,7 +132,7 @@ else()
146132
endif()
147133

148134
install(EXPORT "${PROJECT_NAME}-targets"
149-
DESTINATION "${LIBRARY_DIRECTORY}/${PROJECT_NAME}/cmake/${TARGET_DIR}"
135+
DESTINATION "${CMAKE_INSTALL_LIBDIR}/cmake/${PROJECT_NAME}/${TARGET_DIR}"
150136
NAMESPACE AWS::
151137
COMPONENT Development)
152138

@@ -155,7 +141,7 @@ configure_file("cmake/${PROJECT_NAME}-config.cmake"
155141
@ONLY)
156142

157143
install(FILES "${CMAKE_CURRENT_BINARY_DIR}/${PROJECT_NAME}-config.cmake"
158-
DESTINATION "${LIBRARY_DIRECTORY}/${PROJECT_NAME}/cmake/"
144+
DESTINATION "${CMAKE_INSTALL_LIBDIR}/cmake/${PROJECT_NAME}/"
159145
COMPONENT Development)
160146

161147
include(CTest)

bin/benchmark/CMakeLists.txt

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
project(checksum-profile C)
22

3-
list(APPEND CMAKE_MODULE_PATH "${CMAKE_INSTALL_PREFIX}/lib/cmake")
4-
53
file(GLOB PROFILE_SRC
64
"*.c"
75
)
@@ -25,5 +23,5 @@ install(TARGETS ${PROFILE_PROJECT_NAME}
2523
EXPORT ${PROFILE_PROJECT_NAME}-targets
2624
COMPONENT Runtime
2725
RUNTIME
28-
DESTINATION bin
26+
DESTINATION ${CMAKE_INSTALL_BINDIR}
2927
COMPONENT Runtime)

cmake/AwsSharedLibSetup.cmake

Lines changed: 0 additions & 53 deletions
This file was deleted.

0 commit comments

Comments
 (0)