|
 |
379fd8 |
From 15a06ef27466f42a8dc39823385f0ca655e80b12 Mon Sep 17 00:00:00 2001
|
|
|
379fd8 |
From: =?UTF-8?q?Ale=C5=A1=20Mat=C4=9Bj?= <amatej@redhat.com>
|
|
|
379fd8 |
Date: Sun, 25 Apr 2021 19:47:53 +0200
|
|
|
379fd8 |
Subject: [PATCH 06/19] Revert "Add additional fallback for module resolve"
|
|
|
379fd8 |
|
|
|
379fd8 |
This reverts commit fc98ce725181a77bbe0ab157d40443467612c0c0.
|
|
|
379fd8 |
---
|
|
|
379fd8 |
libdnf/module/ModulePackageContainer.cpp | 32 ++++++++----------------
|
|
|
379fd8 |
1 file changed, 11 insertions(+), 21 deletions(-)
|
|
|
379fd8 |
|
|
|
379fd8 |
diff --git a/libdnf/module/ModulePackageContainer.cpp b/libdnf/module/ModulePackageContainer.cpp
|
|
|
379fd8 |
index ee90056c..e19db08b 100644
|
|
|
379fd8 |
--- a/libdnf/module/ModulePackageContainer.cpp
|
|
|
379fd8 |
+++ b/libdnf/module/ModulePackageContainer.cpp
|
|
|
379fd8 |
@@ -702,33 +702,23 @@ ModulePackageContainer::Impl::moduleSolve(const std::vector<ModulePackage *> & m
|
|
|
379fd8 |
std::vector<std::vector<std::string>> problems;
|
|
|
379fd8 |
auto problemType = ModulePackageContainer::ModuleErrorType::NO_ERROR;
|
|
|
379fd8 |
if (ret) {
|
|
|
379fd8 |
- // Goal run ignor problem in defaults
|
|
|
379fd8 |
problems = goal.describeAllProblemRules(false);
|
|
|
379fd8 |
ret = goal.run(DNF_FORCE_BEST);
|
|
|
379fd8 |
if (ret) {
|
|
|
379fd8 |
- // Goal run ignor problem in defaults and in latest
|
|
|
379fd8 |
- ret = goal.run(DNF_NONE);
|
|
|
379fd8 |
+ // Conflicting modules has to be removed otherwice it could result than one of them will
|
|
|
379fd8 |
+ // be active
|
|
|
379fd8 |
+ auto conflictingPkgs = goal.listConflictPkgs(DNF_PACKAGE_STATE_AVAILABLE);
|
|
|
379fd8 |
+ dnf_sack_add_excludes(moduleSack, conflictingPkgs.get());
|
|
|
379fd8 |
+ ret = goalWeak.run(DNF_NONE);
|
|
|
379fd8 |
if (ret) {
|
|
|
379fd8 |
- // Conflicting modules has to be removed otherwice it could result than one of them will
|
|
|
379fd8 |
- // be active
|
|
|
379fd8 |
- auto conflictingPkgs = goal.listConflictPkgs(DNF_PACKAGE_STATE_AVAILABLE);
|
|
|
379fd8 |
- dnf_sack_add_excludes(moduleSack, conflictingPkgs.get());
|
|
|
379fd8 |
- ret = goalWeak.run(DNF_NONE);
|
|
|
379fd8 |
- if (ret) {
|
|
|
379fd8 |
- auto logger(Log::getLogger());
|
|
|
379fd8 |
- logger->critical("Modularity filtering totally broken\n");
|
|
|
379fd8 |
- problemType = ModulePackageContainer::ModuleErrorType::CANNOT_RESOLVE_MODULES;
|
|
|
379fd8 |
- activatedModules.reset();
|
|
|
379fd8 |
- } else {
|
|
|
379fd8 |
- problemType = ModulePackageContainer::ModuleErrorType::ERROR;
|
|
|
379fd8 |
- Query query(moduleSack, Query::ExcludeFlags::IGNORE_EXCLUDES);
|
|
|
379fd8 |
- goal2name_query(goalWeak, query);
|
|
|
379fd8 |
- activatedModules.reset(new PackageSet(*query.runSet()));
|
|
|
379fd8 |
- }
|
|
|
379fd8 |
+ auto logger(Log::getLogger());
|
|
|
379fd8 |
+ logger->critical("Modularity filtering totally broken\n");
|
|
|
379fd8 |
+ problemType = ModulePackageContainer::ModuleErrorType::CANNOT_RESOLVE_MODULES;
|
|
|
379fd8 |
+ activatedModules.reset();
|
|
|
379fd8 |
} else {
|
|
|
379fd8 |
- problemType = ModulePackageContainer::ModuleErrorType::ERROR_IN_DEFAULTS;
|
|
|
379fd8 |
+ problemType = ModulePackageContainer::ModuleErrorType::ERROR;
|
|
|
379fd8 |
Query query(moduleSack, Query::ExcludeFlags::IGNORE_EXCLUDES);
|
|
|
379fd8 |
- goal2name_query(goal, query);
|
|
|
379fd8 |
+ goal2name_query(goalWeak, query);
|
|
|
379fd8 |
activatedModules.reset(new PackageSet(*query.runSet()));
|
|
|
379fd8 |
}
|
|
|
379fd8 |
} else {
|
|
|
379fd8 |
--
|
|
|
379fd8 |
2.31.1
|
|
|
379fd8 |
|