summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChantry Xavier <shiningxc@gmail.com>2007-07-20 10:22:36 +0200
committerDan McGee <dan@archlinux.org>2007-07-20 16:34:28 +0200
commitb0c064d59b8786a1e169e305a263fc13e2d7ed32 (patch)
tree02349781823247b107aa25cff8140f78c17b8b7a
parent5991e85abb22e32385f6925cf5a6db97a76623f7 (diff)
downloadpacman-b0c064d59b8786a1e169e305a263fc13e2d7ed32.tar.gz
pacman-b0c064d59b8786a1e169e305a263fc13e2d7ed32.tar.xz
libalpm/sync.c : fix DEPENDSONLY flag handling.
I didn't like the patch proposed by Nagy for the sync1002 pactest here: http://www.archlinux.org/pipermail/pacman-dev/2007-July/008971.html So here is another attempt of fixing it. In case of the DEPENDSONLY flag : 1) pass an empty list to resolvedeps instead of the list of targets 2) empty the trans->packages targets list before adding the resolved deps. Signed-off-by: Chantry Xavier <shiningxc@gmail.com>
-rw-r--r--lib/libalpm/sync.c25
1 files changed, 9 insertions, 16 deletions
diff --git a/lib/libalpm/sync.c b/lib/libalpm/sync.c
index c1d2f9e7..7e83120e 100644
--- a/lib/libalpm/sync.c
+++ b/lib/libalpm/sync.c
@@ -392,9 +392,11 @@ int _alpm_sync_prepare(pmtrans_t *trans, pmdb_t *db_local, alpm_list_t *dbs_sync
*data = NULL;
}
- for(i = trans->packages; i; i = i->next) {
- pmsyncpkg_t *sync = i->data;
- list = alpm_list_add(list, sync->pkg);
+ if(!(trans->flags & PM_TRANS_FLAG_DEPENDSONLY)) {
+ for(i = trans->packages; i; i = i->next) {
+ pmsyncpkg_t *sync = i->data;
+ list = alpm_list_add(list, sync->pkg);
+ }
}
if(!(trans->flags & PM_TRANS_FLAG_NODEPS)) {
@@ -411,6 +413,10 @@ int _alpm_sync_prepare(pmtrans_t *trans, pmdb_t *db_local, alpm_list_t *dbs_sync
}
}
+ if((trans->flags & PM_TRANS_FLAG_DEPENDSONLY)) {
+ FREELIST(trans->packages);
+ }
+
for(i = list; i; i = i->next) {
/* add the dependencies found by resolvedeps to the transaction set */
pmpkg_t *spkg = i->data;
@@ -423,19 +429,6 @@ int _alpm_sync_prepare(pmtrans_t *trans, pmdb_t *db_local, alpm_list_t *dbs_sync
trans->packages = alpm_list_add(trans->packages, sync);
_alpm_log(PM_LOG_DEBUG, "adding package %s-%s to the transaction targets",
alpm_pkg_get_name(spkg), alpm_pkg_get_version(spkg));
- } else {
- /* remove the original targets from the list if requested */
- if((trans->flags & PM_TRANS_FLAG_DEPENDSONLY)) {
- void *vpkg;
- pmsyncpkg_t *sync;
-
- _alpm_log(PM_LOG_DEBUG, "removing package %s-%s from the transaction targets",
- alpm_pkg_get_name(spkg), alpm_pkg_get_version(spkg));
-
- sync = _alpm_sync_find(trans->packages, alpm_pkg_get_name(spkg));
- trans->packages = alpm_list_remove(trans->packages, sync, syncpkg_cmp, &vpkg);
- _alpm_sync_free(vpkg);
- }
}
}