From 7b6e44b11b4b419a6af91555270b1ddc981c5a74 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timo=20Ter=C3=A4s?= <timo.teras@iki.fi>
Date: Sat, 19 Mar 2011 14:21:58 +0200
Subject: upgrade: reset world dependencies during traversal

This allows us to get apk-tools dependencies get reset at proper
time in world. As a bonus, it reduces code amount.
---
 src/state.c   | 18 ++++++++++++------
 src/upgrade.c | 20 ++------------------
 2 files changed, 14 insertions(+), 24 deletions(-)

diff --git a/src/state.c b/src/state.c
index 005470c..3a8d3a6 100644
--- a/src/state.c
+++ b/src/state.c
@@ -138,14 +138,20 @@ static struct apk_name_choices *name_choices_new(struct apk_database *db,
 		if (dep->name != name)
 			continue;
 
-		for (j = 0; j < nc->num; ) {
-			if (apk_dep_is_satisfied(dep, nc->pkgs[j])) {
-				j++;
-			} else {
-				nc->pkgs[j] = nc->pkgs[nc->num - 1];
-				nc->num--;
+		if (apk_flags & APK_PREFER_AVAILABLE) {
+			dep->version = apk_blob_atomize(APK_BLOB_NULL);
+			dep->result_mask = APK_DEPMASK_REQUIRE;
+		} else {
+			for (j = 0; j < nc->num; ) {
+				if (apk_dep_is_satisfied(dep, nc->pkgs[j])) {
+					j++;
+				} else {
+					nc->pkgs[j] = nc->pkgs[nc->num - 1];
+					nc->num--;
+				}
 			}
 		}
+		break;
 	}
 
 	return nc;
diff --git a/src/upgrade.c b/src/upgrade.c
index 795e62f..eb51457 100644
--- a/src/upgrade.c
+++ b/src/upgrade.c
@@ -33,21 +33,10 @@ static int upgrade_parse(void *ctx, struct apk_db_options *dbopts,
 int apk_do_self_upgrade(struct apk_database *db, struct apk_state *state)
 {
 	struct apk_dependency dep;
-	int r, i;
+	int r;
 
 	apk_dep_from_blob(&dep, db, APK_BLOB_STR("apk-tools"));
 
-	if (apk_flags & APK_PREFER_AVAILABLE) {
-		for (i = 0; i < db->world->num; i++) {
-			struct apk_dependency *dep0 = &db->world->item[i];
-			if (dep0->name != dep.name)
-				continue;
-			dep0->version = apk_blob_atomize(APK_BLOB_NULL);
-			dep0->result_mask = APK_DEPMASK_REQUIRE;
-			break;
-		}
-	}
-
 	r = apk_state_lock_dependency(state, &dep);
 	if (r != 0 || state->num_changes == 0)
 		return r;
@@ -74,7 +63,6 @@ static int upgrade_main(void *ctx, struct apk_database *db, int argc, char **arg
 {
 	struct apk_state *state = NULL;
 	struct apk_name_array *missing;
-	apk_blob_t *null_atom = apk_blob_atomize(APK_BLOB_NULL);
 	int i, r = 0;
 
 	apk_flags |= APK_UPGRADE;
@@ -92,11 +80,7 @@ static int upgrade_main(void *ctx, struct apk_database *db, int argc, char **arg
 
 	for (i = 0; i < db->world->num; i++) {
 		struct apk_dependency *dep = &db->world->item[i];
-		if (dep->version != null_atom &&
-		    (apk_flags & APK_PREFER_AVAILABLE)) {
-			dep->result_mask = APK_DEPMASK_REQUIRE;
-			dep->version = null_atom;
-		}
+
 		if (dep->name->pkgs->num != 0)
 			r |= apk_state_lock_dependency(state, dep);
 		else
-- 
cgit v1.2.3