From d04cc33f6feb17ede16086080be1ec228a8d0390 Mon Sep 17 00:00:00 2001 From: Timo Teräs Date: Tue, 20 Dec 2022 21:58:41 +0200 Subject: solver: fix queue resolving order It makes sense to not prioritize requirers anymore. It was originally intended to select early packages with multiple constraints seen. However, more important are the constraints that actually limit the choise. fixes #10843 --- src/solver.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/solver.c b/src/solver.c index 3a8f6e6..2a15d7e 100644 --- a/src/solver.c +++ b/src/solver.c @@ -1022,13 +1022,13 @@ static int compare_name_dequeue(const struct apk_name *a, const struct apk_name { int r; - r = (!!a->ss.requirers) - (!!b->ss.requirers); - if (r) return -r; - r = (int)a->priority - (int)b->priority; if (r) return r; r = a->ss.max_dep_chain - b->ss.max_dep_chain; + if (r) return -r; + + r = (!!a->ss.requirers) - (!!b->ss.requirers); return -r; } -- cgit v1.2.3-60-g2f50