From 96c3426bdfb9bb197d55829fa5dc608b21f5f373 Mon Sep 17 00:00:00 2001 From: Neal Gompa Date: Sat, 4 Nov 2023 14:42:27 -0400 Subject: [PATCH] Ensure libsolvext dynamic library dependencies are linked libsolv, when built only build with static link libraries, does not make a fully usable configuration for static libsolvext to be used because the CMake and pkgconfig files do not declare the appropriate libraries to successfully link a static libsolvext that has dynamic library dependencies. This patch works around it by doing the declarations here rather than in libsolv, since it's not easy to fix there, given upstream constraints. --- CMakeLists.txt | 13 +++++++++++++ cmake/modules/FindLZMA.cmake | 25 +++++++++++++++++++++++++ libdnf/CMakeLists.txt | 5 +++++ 3 files changed, 43 insertions(+) create mode 100644 cmake/modules/FindLZMA.cmake diff --git a/CMakeLists.txt b/CMakeLists.txt index 6444c374..15c86b40 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -68,6 +68,19 @@ pkg_check_modules(RPM REQUIRED rpm>=4.15.0) pkg_check_modules(SMARTCOLS REQUIRED smartcols) pkg_check_modules(SQLite3 REQUIRED sqlite3) +# Ensure that static libsolvext dynamic library dependencies are linked in +# libxml2 dependency +find_package (LibXml2 REQUIRED) +# zlib dependency +find_package (ZLIB REQUIRED) +# lzma dependency +find_package (LZMA REQUIRED) +# bzip2 dependency +find_package (BZip2 REQUIRED) +# zstd dependency +find_library (ZSTD_LIBRARY NAMES zstd) +# End static libsolvext dynamic library dependencies + # always enable linking with libdnf utils include_directories(${CMAKE_SOURCE_DIR} libdnf/utils/) diff --git a/cmake/modules/FindLZMA.cmake b/cmake/modules/FindLZMA.cmake new file mode 100644 index 00000000..eb112dff --- /dev/null +++ b/cmake/modules/FindLZMA.cmake @@ -0,0 +1,25 @@ +# - Find lzma +# Find the native LZMA headers and library +# +# LZMA_INCLUDE_DIR - where to find lzma.h, etc. +# LZMA_LIBRARIES - List of libraries when using liblzma. +# LZMA_FOUND - True if liblzma found. + +IF (LZMA_INCLUDE_DIR) + # Already in cache, be silent + SET(LZMA_FIND_QUIETLY TRUE) +ENDIF (LZMA_INCLUDE_DIR) + +FIND_PATH(LZMA_INCLUDE_DIR lzma.h) +FIND_LIBRARY(LZMA_LIBRARY NAMES lzma liblzma) + +# handle the QUIETLY and REQUIRED arguments and set LZMA_FOUND to TRUE if +# all listed variables are TRUE +INCLUDE(FindPackageHandleStandardArgs) +FIND_PACKAGE_HANDLE_STANDARD_ARGS(LZMA DEFAULT_MSG LZMA_LIBRARY LZMA_INCLUDE_DIR) + +IF(LZMA_FOUND) + SET( LZMA_LIBRARIES ${LZMA_LIBRARY} ) +ELSE(LZMA_FOUND) + SET( LZMA_LIBRARIES ) +ENDIF(LZMA_FOUND) diff --git a/libdnf/CMakeLists.txt b/libdnf/CMakeLists.txt index f37d236a..854a50e9 100644 --- a/libdnf/CMakeLists.txt +++ b/libdnf/CMakeLists.txt @@ -81,6 +81,11 @@ target_link_libraries(libdnf ${JSONC_LIBRARIES} ${LIBMODULEMD_LIBRARIES} ${SMARTCOLS_LIBRARIES} + ${LIBXML2_LIBRARIES} + ${ZLIB_LIBRARY} + ${LZMA_LIBRARY} + ${BZIP2_LIBRARY} + ${ZSTD_LIBRARY} ) if(ENABLE_RHSM_SUPPORT) -- 2.41.0