summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Kalnischkies <david@kalnischkies.de>2020-11-05 00:14:32 +0100
committerDavid Kalnischkies <david@kalnischkies.de>2020-11-05 09:40:50 +0100
commit287912d6fc7ab0ab0b5f69a4a5260b5bf53121ba (patch)
tree63c19efefb8d6c07ee9038d3a6b9cbc35e9ea3cb
parente748e07aaab817b3ada3a1b01bf80dbe52852952 (diff)
Rename CMake find_package helpers to avoid developer warnings
| CMake Warning (dev) at /usr/share/cmake-3.18/Modules/FindPackageHandleStandardArgs.cmake:273 (message): | The package name passed to `find_package_handle_standard_args` (Berkeley) | does not match the name of the calling package (BerkeleyDB). This can lead | to problems in calling code that expects `find_package` result variables | (e.g., `_FOUND`) to follow a certain pattern. | Call Stack (most recent call first): | CMake/FindBerkeleyDB.cmake:57 (find_package_handle_standard_args) | CMakeLists.txt:83 (find_package) | This warning is for project developers. Use -Wno-dev to suppress it. And indeed, we checked for BERKLEY_DB_FOUND which was not defined so our HAVE_BDB was not set – just that it is never used, so it wasn't noticed.
-rw-r--r--CMake/FindBerkeley.cmake (renamed from CMake/FindBerkeleyDB.cmake)18
-rw-r--r--CMake/FindGCRYPT.cmake (renamed from CMake/FindGcrypt.cmake)0
-rw-r--r--CMake/FindSECCOMP.cmake (renamed from CMake/FindSeccomp.cmake)0
-rw-r--r--CMake/FindZSTD.cmake (renamed from CMake/FindZstd.cmake)0
-rw-r--r--CMakeLists.txt10
-rw-r--r--ftparchive/CMakeLists.txt6
6 files changed, 17 insertions, 17 deletions
diff --git a/CMake/FindBerkeleyDB.cmake b/CMake/FindBerkeley.cmake
index 34bc3b0d1..a6215d728 100644
--- a/CMake/FindBerkeleyDB.cmake
+++ b/CMake/FindBerkeley.cmake
@@ -1,10 +1,10 @@
# - Try to find Berkeley DB
# Once done this will define
#
-# BERKELEY_DB_FOUND - system has Berkeley DB
-# BERKELEY_DB_INCLUDE_DIRS - the Berkeley DB include directory
-# BERKELEY_DB_LIBRARIES - Link these to use Berkeley DB
-# BERKELEY_DB_DEFINITIONS - Compiler switches required for using Berkeley DB
+# BERKELEY_FOUND - system has Berkeley DB
+# BERKELEY_INCLUDE_DIRS - the Berkeley DB include directory
+# BERKELEY_LIBRARIES - Link these to use Berkeley DB
+# BERKELEY_DEFINITIONS - Compiler switches required for using Berkeley DB
# Copyright (c) 2006, Alexander Dymo, <adymo@kdevelop.org>
# Copyright (c) 2016, Julian Andres Klode <jak@debian.org>
@@ -35,7 +35,7 @@
# We need NO_DEFAULT_PATH here, otherwise CMake helpfully picks up the wrong
# db.h on BSD systems instead of the Berkeley DB one.
-find_path(BERKELEY_DB_INCLUDE_DIRS db.h
+find_path(BERKELEY_INCLUDE_DIRS db.h
${CMAKE_INSTALL_FULL_INCLUDEDIR}/db5
/usr/local/include/db5
/usr/include/db5
@@ -51,9 +51,9 @@ find_path(BERKELEY_DB_INCLUDE_DIRS db.h
NO_DEFAULT_PATH
)
-find_library(BERKELEY_DB_LIBRARIES NAMES db db-5)
+find_library(BERKELEY_LIBRARIES NAMES db db-5)
include(FindPackageHandleStandardArgs)
-find_package_handle_standard_args(Berkeley "Could not find Berkeley DB >= 4.1" BERKELEY_DB_INCLUDE_DIRS BERKELEY_DB_LIBRARIES)
-# show the BERKELEY_DB_INCLUDE_DIRS and BERKELEY_DB_LIBRARIES variables only in the advanced view
-mark_as_advanced(BERKELEY_DB_INCLUDE_DIRS BERKELEY_DB_LIBRARIES)
+find_package_handle_standard_args(Berkeley "Could not find Berkeley DB >= 4.1" BERKELEY_INCLUDE_DIRS BERKELEY_LIBRARIES)
+# show the BERKELEY_INCLUDE_DIRS and BERKELEY_LIBRARIES variables only in the advanced view
+mark_as_advanced(BERKELEY_INCLUDE_DIRS BERKELEY_LIBRARIES)
diff --git a/CMake/FindGcrypt.cmake b/CMake/FindGCRYPT.cmake
index 56bfc9fef..56bfc9fef 100644
--- a/CMake/FindGcrypt.cmake
+++ b/CMake/FindGCRYPT.cmake
diff --git a/CMake/FindSeccomp.cmake b/CMake/FindSECCOMP.cmake
index 5cfd13a37..5cfd13a37 100644
--- a/CMake/FindSeccomp.cmake
+++ b/CMake/FindSECCOMP.cmake
diff --git a/CMake/FindZstd.cmake b/CMake/FindZSTD.cmake
index 68118049b..68118049b 100644
--- a/CMake/FindZstd.cmake
+++ b/CMake/FindZSTD.cmake
diff --git a/CMakeLists.txt b/CMakeLists.txt
index f4afce7ea..51f0c25d9 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -80,8 +80,8 @@ add_optional_compile_options(Wsuggest-override)
add_optional_compile_options(Werror=suggest-override)
add_optional_compile_options(Werror=return-type)
# apt-ftparchive dependencies
-find_package(BerkeleyDB REQUIRED)
-if (BERKELEY_DB_FOUND)
+find_package(Berkeley REQUIRED)
+if (BERKELEY_FOUND)
set(HAVE_BDB 1)
endif()
@@ -113,7 +113,7 @@ if (LZ4_FOUND)
set(HAVE_LZ4 1)
endif()
-find_package(Zstd)
+find_package(ZSTD)
if (ZSTD_FOUND)
set(HAVE_ZSTD 1)
endif()
@@ -129,12 +129,12 @@ if (SYSTEMD_FOUND)
set(HAVE_SYSTEMD 1)
endif()
-find_package(Seccomp)
+find_package(SECCOMP)
if (SECCOMP_FOUND)
set(HAVE_SECCOMP 1)
endif()
-find_package(Gcrypt REQUIRED)
+find_package(GCRYPT REQUIRED)
# Mount()ing and stat()ing and friends
check_symbol_exists(statfs sys/vfs.h HAVE_VFS_H)
diff --git a/ftparchive/CMakeLists.txt b/ftparchive/CMakeLists.txt
index 9aa6606f1..4af250474 100644
--- a/ftparchive/CMakeLists.txt
+++ b/ftparchive/CMakeLists.txt
@@ -1,4 +1,4 @@
-include_directories(${BERKELEY_DB_INCLUDE_DIRS})
+include_directories(${BERKELEY_INCLUDE_DIRS})
# Definition of the C++ files used to build the program - note that this
# is expanded at CMake time, so you have to rerun cmake if you add or remove
# a file (you can just run cmake . in the build directory)
@@ -6,8 +6,8 @@ file(GLOB_RECURSE source "*.cc")
add_executable(apt-ftparchive ${source})
# Link the executables against the libraries
-target_include_directories(apt-ftparchive PRIVATE ${BERKELEY_DB_INCLUDE_DIRS})
-target_link_libraries(apt-ftparchive apt-pkg apt-private ${BERKELEY_DB_LIBRARIES})
+target_include_directories(apt-ftparchive PRIVATE ${BERKELEY_INCLUDE_DIRS})
+target_link_libraries(apt-ftparchive apt-pkg apt-private ${BERKELEY_LIBRARIES})
# Install the executables
install(TARGETS apt-ftparchive RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR})