diff options
| author | Andrew Gregory <andrew.gregory.8@gmail.com> | 2013-12-11 17:58:28 +0100 |
|---|---|---|
| committer | Allan McRae <allan@archlinux.org> | 2013-12-19 06:19:52 +0100 |
| commit | 2f8be5f8db1e157d3bcdac763b79c7989567ca6d (patch) | |
| tree | 434e2718fe11193383c5cf737b033c718cf7f277 /lib/libalpm/trans.c | |
| parent | 714609639f9a316e037252d31437505520a7966f (diff) | |
| download | pacman-2f8be5f8db1e157d3bcdac763b79c7989567ca6d.tar.gz pacman-2f8be5f8db1e157d3bcdac763b79c7989567ca6d.tar.xz | |
trans_prepare: always sort trans->remove by deps
Packages can be removed during a sync transaction either directly or
due to conflicts and need to be sorted.
Signed-off-by: Andrew Gregory <andrew.gregory.8@gmail.com>
Signed-off-by: Allan McRae <allan@archlinux.org>
Diffstat (limited to 'lib/libalpm/trans.c')
| -rw-r--r-- | lib/libalpm/trans.c | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/lib/libalpm/trans.c b/lib/libalpm/trans.c index a795a1fa..6ace823e 100644 --- a/lib/libalpm/trans.c +++ b/lib/libalpm/trans.c @@ -39,6 +39,7 @@ #include "remove.h" #include "sync.h" #include "alpm.h" +#include "deps.h" /** \addtogroup alpm_trans Transaction Functions * @brief Functions to manipulate libalpm transactions @@ -134,6 +135,21 @@ int SYMEXPORT alpm_trans_prepare(alpm_handle_t *handle, alpm_list_t **data) } } + + if(!(trans->flags & ALPM_TRANS_FLAG_NODEPS)) { + _alpm_log(handle, ALPM_LOG_DEBUG, "sorting by dependencies\n"); + if(trans->add) { + alpm_list_t *add_orig = trans->add; + trans->add = _alpm_sortbydeps(handle, add_orig, trans->remove, 0); + alpm_list_free(add_orig); + } + if(trans->remove) { + alpm_list_t *rem_orig = trans->remove; + trans->remove = _alpm_sortbydeps(handle, rem_orig, NULL, 1); + alpm_list_free(rem_orig); + } + } + trans->state = STATE_PREPARED; return 0; |