From 9482a9d7d49597da1f28b21642ba602e864b91df 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 d7386d2..68aec53 100644 --- a/src/solver.c +++ b/src/solver.c @@ -1023,13 +1023,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-70-g09d2