From 0565e9bb050ab979f6b8bf7920c5820d8abaf702 Mon Sep 17 00:00:00 2001
From: Timo Teräs <timo.teras@iki.fi>
Date: Mon, 12 Nov 2018 15:00:14 +0200
Subject: fix orphan package handling for certain provides cases
---
src/solver.c | 32 +++++++++++++++++---------------
1 file changed, 17 insertions(+), 15 deletions(-)
(limited to 'src')
diff --git a/src/solver.c b/src/solver.c
index e1d8e2b..e10cf8b 100644
--- a/src/solver.c
+++ b/src/solver.c
@@ -817,6 +817,17 @@ static void cset_check_removal_by_iif(struct apk_solver_state *ss, struct apk_na
}
}
+static void cset_gen_name_remove_orphan(struct apk_solver_state *ss, struct apk_name *name)
+{
+ struct apk_package *pkg = name->ss.chosen.pkg;
+
+ if (name->ss.in_changeset) return;
+ name->ss.in_changeset = 1;
+
+ if ((!pkg || pkg->name != name) && name->ss.installed_pkg)
+ cset_gen_name_remove(ss, name->ss.installed_pkg);
+}
+
static void cset_gen_name_change(struct apk_solver_state *ss, struct apk_name *name)
{
struct apk_name **pname;
@@ -825,24 +836,15 @@ static void cset_gen_name_change(struct apk_solver_state *ss, struct apk_name *n
if (name->ss.in_changeset) return;
- pkg = name->ss.chosen.pkg;
- if (pkg == NULL || pkg->name != name) {
- /* Original package dependency name was orphaned, emit a removal.
- * See cset_gen_name_remove() for more details. */
- opkg = name->ss.installed_pkg;
- if (opkg) cset_gen_name_remove(ss, opkg);
- name->ss.in_changeset = 1;
-
- /* If a replacement is not provided, then we're done here. */
- if (pkg == NULL)
- return;
- }
- if (pkg->ss.in_changeset) return;
+ cset_gen_name_remove_orphan(ss, name);
+ pkg = name->ss.chosen.pkg;
+ if (!pkg || pkg->ss.in_changeset) return;
pkg->ss.in_changeset = 1;
- pkg->name->ss.in_changeset = 1;
+
+ cset_gen_name_remove_orphan(ss, pkg->name);
foreach_array_item(d, pkg->provides)
- d->name->ss.in_changeset = 1;
+ cset_gen_name_remove_orphan(ss, d->name);
opkg = pkg->name->ss.installed_pkg;
if (opkg) {
--
cgit v1.2.3-70-g09d2