diff options
author | Xavier Chantry <shiningxc@gmail.com> | 2008-07-25 02:02:36 +0200 |
---|---|---|
committer | Dan McGee <dan@archlinux.org> | 2008-07-24 22:16:29 -0500 |
commit | dfae7bdd52476673424f8020befef166cf95f3fc (patch) | |
tree | dfa54d7df22f17f29d27673cf416a81b46bf9acb /src/pacman/sync.c | |
parent | e6fb229534332396363f69bdb46081f45271bc53 (diff) |
src/pacman : rework the display_targets function.
We had a lot of duplicated code here. The code handling the showsize option
needed to be there three times :
1) for install part of -S
2) for remove part of -S (conflict removal)
3) for -R
This patch introduce a new display_targets(pkglist, install) function which
can handle the 3 cases above. We pass install == 1 for case 1), and install
== 0 for case 2) and 3).
Now we can finally get the benefit of an old patch which handled the
ShowSize option consistently in the 3 cases above, without an awful lot of
duplicated code :
http://www.archlinux.org/pipermail/pacman-dev/2008-January/011029.html
Signed-off-by: Xavier Chantry <shiningxc@gmail.com>
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'src/pacman/sync.c')
-rw-r--r-- | src/pacman/sync.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/pacman/sync.c b/src/pacman/sync.c index 87bd5bec..8d0c529a 100644 --- a/src/pacman/sync.c +++ b/src/pacman/sync.c @@ -669,7 +669,7 @@ static int sync_trans(alpm_list_t *targets) if(!(alpm_trans_get_flags() & PM_TRANS_FLAG_PRINTURIS)) { int confirm; - display_targets(packages, db_local); + display_synctargets(packages); printf("\n"); if(config->op_s_downloadonly) { |