diff options
author | Allan McRae <allan@archlinux.org> | 2011-09-29 21:13:22 +1000 |
---|---|---|
committer | Dan McGee <dan@archlinux.org> | 2011-09-30 10:48:01 -0500 |
commit | 6dfb9d35f8601c3752dc5bd0e8558c48f41a8215 (patch) | |
tree | 5a55d27bbe7c733371a8355283cb2642ddfbcd0e | |
parent | 83c05e71bc6e614f6f3d190d5e752a0c5239a2d1 (diff) |
Remove redundant transaction size output
Printing all of "Installed", "Removed" and "Net Upgrade" sizes is
redundant as the difference of the first two is the last. Instead,
only print "Installed Size" and "Net Upgrade Size" when both the
installed and removed are non-zero.
This results in the following output in the following cases:
- package installation only: Installed Size
- package removal only: Removed Size
- package installation involving replacement: Installed + Net Upgrade Size
- package upgrade: Installed + Net Upgrade Size
- combination upgrade and installation: Installed + Net Upgrade Size
Download Size remains outputted whenever something is downloaded.
Signed-off-by: Allan McRae <allan@archlinux.org>
Signed-off-by: Dan McGee <dan@archlinux.org>
-rw-r--r-- | src/pacman/util.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/pacman/util.c b/src/pacman/util.c index c3836a5e..79fb54dc 100644 --- a/src/pacman/util.c +++ b/src/pacman/util.c @@ -892,7 +892,7 @@ static void _display_targets(alpm_list_t *targets, int verbose) size = humanize_size(isize, 'M', &label); printf(_("Total Installed Size: %.2f %s\n"), size, label); } - if(rsize > 0) { + if(rsize > 0 && isize == 0) { size = humanize_size(rsize, 'M', &label); printf(_("Total Removed Size: %.2f %s\n"), size, label); } |