From 31b95e3c242654f8a7949e8d06df20101668b5a7 Mon Sep 17 00:00:00 2001 From: Ross Burton Date: Mon, 22 Sep 2025 15:20:28 +0100 Subject: [PATCH] libdnf: don't depend on libcheck Apply a patch to make libcheck optional, so we don't need to depend on it as we don't install the tests. Signed-off-by: Ross Burton Signed-off-by: Mathieu Dubois-Briand Signed-off-by: Richard Purdie --- .../libdnf/libdnf/optional-check.patch | 52 +++++++++++++++++++ meta/recipes-devtools/libdnf/libdnf_0.74.0.bb | 4 +- 2 files changed, 54 insertions(+), 2 deletions(-) create mode 100644 meta/recipes-devtools/libdnf/libdnf/optional-check.patch diff --git a/meta/recipes-devtools/libdnf/libdnf/optional-check.patch b/meta/recipes-devtools/libdnf/libdnf/optional-check.patch new file mode 100644 index 0000000000..9228b0560f --- /dev/null +++ b/meta/recipes-devtools/libdnf/libdnf/optional-check.patch @@ -0,0 +1,52 @@ +From a4abd42a6b92f6aa16490c0f482bf08c4a6c2864 Mon Sep 17 00:00:00 2001 +From: Ross Burton +Date: Thu, 18 Sep 2025 11:23:33 +0100 +Subject: [PATCH] Move libcheck dependency to tests/ + +If we're not building the tests then there's no point in depending on +libcheck, so move the pkg_check_modules() call to tests/CMakeLists. + +Upstream-Status: Backport [https://github.com/rpm-software-management/libdnf/commit/6a127aec78d2ef837776e0e5a8e3636101a54ab0] +Signed-off-by: Ross Burton +--- + CMakeLists.txt | 1 - + tests/CMakeLists.txt | 7 +++---- + 2 files changed, 3 insertions(+), 5 deletions(-) + +diff --git a/CMakeLists.txt b/CMakeLists.txt +index a489ea73..d1969899 100644 +--- a/CMakeLists.txt ++++ b/CMakeLists.txt +@@ -59,7 +59,6 @@ find_package(LibSolv 0.7.21 REQUIRED COMPONENTS ext) + + + # build dependencies via pkg-config +-pkg_check_modules(CHECK REQUIRED check) + pkg_check_modules(GLIB REQUIRED gio-unix-2.0>=2.46.0) + include_directories(${GLIB_INCLUDE_DIRS}) + pkg_check_modules(JSONC REQUIRED json-c) +diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt +index e4909682..78743f4e 100644 +--- a/tests/CMakeLists.txt ++++ b/tests/CMakeLists.txt +@@ -1,3 +1,6 @@ ++pkg_check_modules(CHECK REQUIRED check) ++pkg_check_modules(CPPUNIT REQUIRED cppunit) ++ + add_subdirectory(libdnf/conf) + add_subdirectory(libdnf/module/modulemd) + add_subdirectory(libdnf/module) +@@ -7,10 +10,6 @@ add_subdirectory(libdnf/sack) + add_subdirectory(hawkey) + add_subdirectory(libdnf) + +- +- +-pkg_check_modules(CPPUNIT REQUIRED cppunit) +- + set(LIBDNF_TEST_SOURCES + ${LIBDNF_TEST_SOURCES} + ${CMAKE_CURRENT_SOURCE_DIR}/run_tests.cpp +-- +2.43.0 + diff --git a/meta/recipes-devtools/libdnf/libdnf_0.74.0.bb b/meta/recipes-devtools/libdnf/libdnf_0.74.0.bb index 6ac518564b..0dce8dc183 100644 --- a/meta/recipes-devtools/libdnf/libdnf_0.74.0.bb +++ b/meta/recipes-devtools/libdnf/libdnf_0.74.0.bb @@ -11,12 +11,13 @@ SRC_URI = "git://github.com/rpm-software-management/libdnf;branch=dnf-4-master;p file://enable_test_data_dir_set.patch \ file://0001-drop-FindPythonInstDir.cmake.patch \ file://armarch.patch \ + file://optional-check.patch \ " SRCREV = "91a0bf9aada36a722855051526f012e0b5ab1af9" UPSTREAM_CHECK_GITTAGREGEX = "(?P(?!4\.90)\d+(\.\d+)+)" -DEPENDS = "glib-2.0 libsolv libcheck librepo rpm gtk-doc libmodulemd json-c swig-native util-linux" +DEPENDS = "glib-2.0 libsolv librepo rpm gtk-doc libmodulemd json-c swig-native util-linux" inherit gtk-doc gobject-introspection cmake pkgconfig setuptools3-base @@ -31,4 +32,3 @@ EXTRA_OECMAKE:append:class-nativesdk = " -DWITH_GIR=OFF" BBCLASSEXTEND = "native nativesdk" SKIP_RECIPE[libdnf] ?= "${@bb.utils.contains('PACKAGE_CLASSES', 'package_rpm', '', 'Does not build without package_rpm in PACKAGE_CLASSES due disabled rpm support in libsolv', d)}" - -- 2.47.3