Hello community, here is the log from the commit of package libdnf for openSUSE:Factory checked in at 2019-04-28 20:12:02 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/libdnf (Old) and /work/SRC/openSUSE:Factory/.libdnf.new.5536 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Package is "libdnf" Sun Apr 28 20:12:02 2019 rev:9 rq:698443 version:0.31.0 Changes: -------- --- /work/SRC/openSUSE:Factory/libdnf/libdnf.changes 2019-04-01 12:39:54.793959710 +0200 +++ /work/SRC/openSUSE:Factory/.libdnf.new.5536/libdnf.changes 2019-04-28 20:12:17.986461496 +0200 @@ -1,0 +2,16 @@ +Fri Apr 26 18:53:23 UTC 2019 - Neal Gompa <ngompa13@gmail.com> + +- Add patch for SLE to temporarily drop support for Module advisories + * Patch: libdnf-0.31.0-Revert-support-for-Module-advisories.patch + +------------------------------------------------------------------- +Fri Apr 26 10:58:12 UTC 2019 - Neal Gompa <ngompa13@gmail.com> + +- Update to version 0.31.0 + + Installroot now requires absolute path + + Support "_none_" value for repo option "proxy" (rh#1680272) + + Add support for Module advisories + + Add support for xml:base attribute from primary.xml (rh#1691315) + + Fix detection of Platform ID for modules (rh#1688462) + +------------------------------------------------------------------- Old: ---- libdnf-0.28.1.tar.gz New: ---- libdnf-0.31.0-Revert-support-for-Module-advisories.patch libdnf-0.31.0.tar.gz ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ libdnf.spec ++++++ --- /var/tmp/diff_new_pack.I7Z0rH/_old 2019-04-28 20:12:20.114460172 +0200 +++ /var/tmp/diff_new_pack.I7Z0rH/_new 2019-04-28 20:12:20.162460143 +0200 @@ -16,8 +16,12 @@ # Please submit bugfixes or comments via https://bugs.opensuse.org/ # +%if (0%{?sle_version} && 0%{?sle_version} < 150200) + %global libsolv_version 0.7.3 +%else + %global libsolv_version 0.7.4 +%endif -%global libsolv_version 0.6.34 %global libmodulemd_version 1.6.1 %global librepo_version 1.9.6 %global swig_version 3.0.12 @@ -33,7 +37,7 @@ %define devname %{name}-devel Name: libdnf -Version: 0.28.1 +Version: 0.31.0 Release: 0 Summary: Library providing C and Python APIs atop libsolv License: LGPL-2.1-or-later @@ -46,6 +50,10 @@ # PATCH-FIX-OPENSUSE: Switch default reposdir to /etc/dnf/repos.d Patch1001: libdnf-0.28.1-Switch-default-reposdir-to-etc-dnf-repos.d.patch +# PATCH-FIX-SLE: Revert support for module advisories, since that requires libsolv 0.7.4 +## Drop when bsc#1133527 is fixed +Patch2001: libdnf-0.31.0-Revert-support-for-Module-advisories.patch + BuildRequires: cmake BuildRequires: gcc BuildRequires: gcc-c++ @@ -145,6 +153,11 @@ %prep %autosetup -p1 +%if (0%{?sle_version} && 0%{?sle_version} > 150100) || 0%{?suse_version} > 1500 +# Revert this patch for SLE 15 >SP1 and SUSE Linux >15 +%patch2001 -R -p1 +%endif + # Fix sphinx-build run... sed -e "s/sphinx-build-3/sphinx-build-%{python3_version}/" -i docs/hawkey/CMakeLists.txt ++++++ libdnf-0.31.0-Revert-support-for-Module-advisories.patch ++++++

From 4e296a4d56595a10e4532b0e9bbe5a0d6eb75d01 Mon Sep 17 00:00:00 2001 From: Neal Gompa <ngompa13@gmail.com> Date: Fri, 26 Apr 2019 07:41:04 -0400 Subject: [PATCH] Revert support for Module advisories

SUSE Linux Enterprise 15 SP1 (and thus openSUSE Leap 15.1) do not have libsolv 0.7.4, so handling advisories for modules is not supported. This reverts the following commits (in order): * 33a73d7b54cabd20347aac9d59bf9cb1e308f975 * defb481a340cc3e50fde69e114373e66ada74835 --- libdnf.spec | 4 +- libdnf/hy-package.cpp | 7 +- libdnf/sack/CMakeLists.txt | 1 - libdnf/sack/advisory.cpp | 21 ------ libdnf/sack/advisory.hpp | 2 - libdnf/sack/advisorymodule.cpp | 113 --------------------------------- libdnf/sack/advisorymodule.hpp | 58 ----------------- libdnf/sack/query.cpp | 16 ++--- libdnf/utils/utils.cpp | 28 -------- libdnf/utils/utils.hpp | 4 -- 10 files changed, 8 insertions(+), 246 deletions(-) delete mode 100644 libdnf/sack/advisorymodule.cpp delete mode 100644 libdnf/sack/advisorymodule.hpp diff --git a/libdnf.spec b/libdnf.spec index aa09f497..e14d7dd6 100644 --- a/libdnf.spec +++ b/libdnf.spec @@ -1,7 +1,7 @@ -%global libsolv_version 0.7.4-1 +%global libsolv_version 0.6.35-1 %global libmodulemd_version 1.6.1 %global librepo_version 1.9.5 -%global dnf_conflict 4.2.5 +%global dnf_conflict 4.2.3 %global swig_version 3.0.12 %bcond_with valgrind diff --git a/libdnf/hy-package.cpp b/libdnf/hy-package.cpp index 564bc464..833e082c 100644 --- a/libdnf/hy-package.cpp +++ b/libdnf/hy-package.cpp @@ -29,7 +29,6 @@ * See also: #DnfContext */ -#include "libdnf/utils/utils.hpp" #include <algorithm> #include <ctime> @@ -1016,11 +1015,7 @@ dnf_package_get_advisories(DnfPackage *pkg, int cmp_type) (cmp < 0 && (cmp_type & HY_LT)) || (cmp == 0 && (cmp_type & HY_EQ))) { advisory = dnf_advisory_new(sack, di.solvid); - if (libdnf::isAdvisoryApplicable(*advisory, sack)) { - g_ptr_array_add(advisorylist, advisory); - } else { - dnf_advisory_free(advisory); - } + g_ptr_array_add(advisorylist, advisory); dataiterator_skip_solvable(&di); } } diff --git a/libdnf/sack/CMakeLists.txt b/libdnf/sack/CMakeLists.txt index 40d3373c..5e8d5e25 100644 --- a/libdnf/sack/CMakeLists.txt +++ b/libdnf/sack/CMakeLists.txt @@ -1,7 +1,6 @@ set(SACK_SOURCES ${SACK_SOURCES} ${CMAKE_CURRENT_SOURCE_DIR}/advisory.cpp - ${CMAKE_CURRENT_SOURCE_DIR}/advisorymodule.cpp ${CMAKE_CURRENT_SOURCE_DIR}/advisorypkg.cpp ${CMAKE_CURRENT_SOURCE_DIR}/advisoryref.cpp ${CMAKE_CURRENT_SOURCE_DIR}/packageset.cpp diff --git a/libdnf/sack/advisory.cpp b/libdnf/sack/advisory.cpp index bb9e7f40..d006bcc5 100644 --- a/libdnf/sack/advisory.cpp +++ b/libdnf/sack/advisory.cpp @@ -24,7 +24,6 @@ #include "advisory.hpp" #include "advisorypkg.hpp" -#include "advisorymodule.hpp" #include "advisoryref.hpp" #include "../dnf-advisory-private.hpp" #include "../dnf-advisoryref.h" @@ -134,26 +133,6 @@ Advisory::getPackages(std::vector<AdvisoryPkg> & pkglist, bool withFilemanes) co dataiterator_free(&di); } -std::vector<AdvisoryModule> Advisory::getModules() const -{ - std::vector<AdvisoryModule> moduleList; - Dataiterator di; - Pool *pool = dnf_sack_get_pool(sack); - - dataiterator_init(&di, pool, 0, advisory, UPDATE_MODULE, 0, 0); - while (dataiterator_step(&di)) { - dataiterator_setpos(&di); - Id name = pool_lookup_id(pool, SOLVID_POS, UPDATE_MODULE_NAME); - Id stream = pool_lookup_id(pool, SOLVID_POS, UPDATE_MODULE_STREAM); - Id version = pool_lookup_id(pool, SOLVID_POS, UPDATE_MODULE_VERSION); - Id context = pool_lookup_id(pool, SOLVID_POS, UPDATE_MODULE_CONTEXT); - Id arch = pool_lookup_id(pool, SOLVID_POS, UPDATE_MODULE_ARCH); - moduleList.emplace_back(sack, advisory, name, stream, version, context, arch); - } - dataiterator_free(&di); - return moduleList; -} - void Advisory::getReferences(std::vector<AdvisoryRef> & reflist) const { diff --git a/libdnf/sack/advisory.hpp b/libdnf/sack/advisory.hpp index 5aa8dc60..37712573 100644 --- a/libdnf/sack/advisory.hpp +++ b/libdnf/sack/advisory.hpp @@ -33,7 +33,6 @@ namespace libdnf { struct AdvisoryPkg; -struct AdvisoryModule; struct Advisory { public: @@ -43,7 +42,6 @@ public: DnfAdvisoryKind getKind() const; const char *getName() const; void getPackages(std::vector<AdvisoryPkg> & pkglist, bool withFilemanes = true) const; - std::vector<AdvisoryModule> getModules() const; void getReferences(std::vector<AdvisoryRef> & reflist) const; const char *getRights() const; const char *getSeverity() const; diff --git a/libdnf/sack/advisorymodule.cpp b/libdnf/sack/advisorymodule.cpp deleted file mode 100644 index a209b5f1..00000000 --- a/libdnf/sack/advisorymodule.cpp +++ /dev/null @@ -1,113 +0,0 @@ -/* - * Copyright (C) 2019 Red Hat, Inc. - * - * Licensed under the GNU Lesser General Public License Version 2.1 - * - * This library is free software; you can redistribute it and/or - * modify it under the terms of the GNU Lesser General Public - * License as published by the Free Software Foundation; either - * version 2.1 of the License, or (at your option) any later version. - * - * This library is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - * Lesser General Public License for more details. - * - * You should have received a copy of the GNU Lesser General Public - * License along with this library; if not, write to the Free Software - * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA - */ - -#include <string> - -#include <solv/poolid.h> - -#include "advisory.hpp" -#include "advisorymodule.hpp" -#include "../dnf-sack-private.hpp" - -namespace libdnf { - -class AdvisoryModule::Impl { -private: - friend AdvisoryModule; - DnfSack *sack; - Id advisory; - Id name; - Id stream; - Id version; - Id context; - Id arch; -}; - -AdvisoryModule::AdvisoryModule(DnfSack *sack, Id advisory, Id name, Id stream, Id version, Id context, Id arch) : pImpl(new Impl) -{ - pImpl->sack = sack; - pImpl->advisory = advisory; - pImpl->name = name; - pImpl->stream = stream; - pImpl->version = version; - pImpl->context = context; - pImpl->arch = arch; -} -AdvisoryModule::AdvisoryModule(const AdvisoryModule & src) : pImpl(new Impl) { *pImpl = *src.pImpl; } -AdvisoryModule::AdvisoryModule(AdvisoryModule && src) : pImpl(new Impl) { pImpl.swap(src.pImpl); } -AdvisoryModule::~AdvisoryModule() = default; - -AdvisoryModule & AdvisoryModule::operator=(const AdvisoryModule & src) { *pImpl = *src.pImpl; return *this; } - -AdvisoryModule & -AdvisoryModule::operator=(AdvisoryModule && src) noexcept -{ - pImpl.swap(src.pImpl); - return *this; -} - -bool -AdvisoryModule::nsvcaEQ(AdvisoryModule & other) -{ - return other.pImpl->name == pImpl->name && - other.pImpl->stream == pImpl->stream && - other.pImpl->version == pImpl->version && - other.pImpl->context == pImpl->context && - other.pImpl->arch == pImpl->arch; -} - -Advisory * AdvisoryModule::getAdvisory() const -{ - return new Advisory(pImpl->sack, pImpl->advisory); -} - -const char * -AdvisoryModule::getName() const -{ - return pool_id2str(dnf_sack_get_pool(pImpl->sack), pImpl->name); -} - -const char * -AdvisoryModule::getStream() const -{ - return pool_id2str(dnf_sack_get_pool(pImpl->sack), pImpl->stream); -} - -const char * -AdvisoryModule::getVersion() const -{ - return pool_id2str(dnf_sack_get_pool(pImpl->sack), pImpl->version); -} - -const char * -AdvisoryModule::getContext() const -{ - return pool_id2str(dnf_sack_get_pool(pImpl->sack), pImpl->context); -} - -const char * -AdvisoryModule::getArch() const -{ - return pool_id2str(dnf_sack_get_pool(pImpl->sack), pImpl->arch); -} - -DnfSack * AdvisoryModule::getSack() { return pImpl->sack; } - -} diff --git a/libdnf/sack/advisorymodule.hpp b/libdnf/sack/advisorymodule.hpp deleted file mode 100644 index a36bee03..00000000 --- a/libdnf/sack/advisorymodule.hpp +++ /dev/null @@ -1,58 +0,0 @@ -/* - * Copyright (C) 2019 Red Hat, Inc. - * - * Licensed under the GNU Lesser General Public License Version 2.1 - * - * This library is free software; you can redistribute it and/or - * modify it under the terms of the GNU Lesser General Public - * License as published by the Free Software Foundation; either - * version 2.1 of the License, or (at your option) any later version. - * - * This library is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - * Lesser General Public License for more details. - * - * You should have received a copy of the GNU Lesser General Public - * License along with this library; if not, write to the Free Software - * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA - */ - - -#ifndef __ADVISORY_MODULE_HPP -#define __ADVISORY_MODULE_HPP - -#include "../dnf-types.h" -#include "advisory.hpp" - -#include <memory> - -#include <solv/pooltypes.h> - - -namespace libdnf { - -struct AdvisoryModule { -public: - AdvisoryModule(DnfSack *sack, Id advisory, Id name, Id stream, Id version, Id context, Id arch); - AdvisoryModule(const AdvisoryModule & src); - AdvisoryModule(AdvisoryModule && src); - ~AdvisoryModule(); - AdvisoryModule & operator=(const AdvisoryModule & src); - AdvisoryModule & operator=(AdvisoryModule && src) noexcept; - bool nsvcaEQ(AdvisoryModule & other); - Advisory * getAdvisory() const; - const char * getName() const; - const char * getStream() const; - const char * getVersion() const; - const char * getContext() const; - const char * getArch() const; - DnfSack * getSack(); -private: - class Impl; - std::unique_ptr<Impl> pImpl; -}; - -} - -#endif /* __ADVISORY_MODULE_HPP */ diff --git a/libdnf/sack/query.cpp b/libdnf/sack/query.cpp index 99406480..256b114f 100644 --- a/libdnf/sack/query.cpp +++ b/libdnf/sack/query.cpp @@ -18,8 +18,6 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ -#include "libdnf/utils/utils.hpp" - #include <algorithm> #include <assert.h> #include <fnmatch.h> @@ -1532,9 +1530,7 @@ Query::Impl::filterAdvisory(const Filter & f, Map *m, int keyname) eq = false; } if (eq) { - if (isAdvisoryApplicable(advisory, sack)) { - advisory.getPackages(pkgs, false); - } + advisory.getPackages(pkgs, false); break; } } @@ -2079,8 +2075,7 @@ void Query::getAdvisoryPkgs(int cmpType, std::vector<AdvisoryPkg> & advisoryPkgs) { apply(); - auto sack = pImpl->sack; - Pool *pool = dnf_sack_get_pool(sack); + Pool *pool = dnf_sack_get_pool(pImpl->sack); std::vector<AdvisoryPkg> pkgs; Dataiterator di; auto resultPset = pImpl->result.get(); @@ -2089,10 +2084,9 @@ Query::getAdvisoryPkgs(int cmpType, std::vector<AdvisoryPkg> & advisoryPkgs) dataiterator_init(&di, pool, 0, 0, 0, 0, 0); dataiterator_prepend_keyname(&di, UPDATE_COLLECTION); while (dataiterator_step(&di)) { - Advisory advisory(sack, di.solvid); - if (isAdvisoryApplicable(advisory, sack)) { - advisory.getPackages(pkgs); - } + Advisory advisory(pImpl->sack, di.solvid); + + advisory.getPackages(pkgs); dataiterator_skip_solvable(&di); } dataiterator_free(&di); diff --git a/libdnf/utils/utils.cpp b/libdnf/utils/utils.cpp index 458929e3..a12daa41 100644 --- a/libdnf/utils/utils.cpp +++ b/libdnf/utils/utils.cpp @@ -1,6 +1,4 @@ #include "utils.hpp" -#include "libdnf/dnf-sack-private.hpp" -#include "libdnf/sack/advisorymodule.hpp" #include <tinyformat/tinyformat.hpp> @@ -21,32 +19,6 @@ extern "C" { namespace libdnf { -bool isAdvisoryApplicable(libdnf::Advisory & advisory, DnfSack * sack) -{ - auto moduleContainer = dnf_sack_get_module_container(sack); - if (!moduleContainer) { - return true; - } - auto moduleAdvisories = advisory.getModules(); - if (moduleAdvisories.empty()) { - return true; - } - for (auto & moduleAdvisory: moduleAdvisories) { - if (const char * name = moduleAdvisory.getName()) { - if (const char * stream = moduleAdvisory.getStream()) { - try { - if (moduleContainer->isEnabled(name, stream)) { - return true; - } - } catch (std::out_of_range &) { - continue; - } - } - } - } - return false; -} - namespace string { std::vector<std::string> split(const std::string &source, const char *delimiter, int maxSplit) diff --git a/libdnf/utils/utils.hpp b/libdnf/utils/utils.hpp index a653047f..326e4a97 100644 --- a/libdnf/utils/utils.hpp +++ b/libdnf/utils/utils.hpp @@ -7,8 +7,6 @@ #define DIGITS "0123456789" #define REPOID_CHARS ASCII_LETTERS DIGITS "-_.:" -#include "libdnf/sack/advisory.hpp" - #include <functional> #include <string> #include <vector> @@ -36,8 +34,6 @@ private: std::function<void()> func; }; -bool isAdvisoryApplicable(Advisory & advisory, DnfSack * sack); - namespace string { inline std::string fromCstring(const char * cstring) { return cstring ? cstring : ""; } std::vector<std::string> split(const std::string &source, const char *delimiter, int maxSplit = -1); -- 2.20.1 ++++++ libdnf-0.28.1.tar.gz -> libdnf-0.31.0.tar.gz ++++++ ++++ 3562 lines of diff (skipped)