On Mon, 2011-09-19 at 09:26 +0200, Zdeněk Pavlas wrote:
> All 'presto' cares about are updates, the other message
> (Presto reduced the update size) is explicit in this.
>
> So maybe just change the message?
> ---
> yum-presto/presto.py | 5 ++++-
> 1 files changed, 4 insertions(+), 1 deletions(-)
>
> diff --git a/yum-presto/presto.py b/yum-presto/presto.py
> index 169e938..fe81143 100644
> --- a/yum-presto/presto.py
> +++ b/yum-presto/presto.py
> @@ -588,8 +588,11 @@ def predownload_hook(conduit):
> repos = set()
> reposize = {}
> download_pkgs = []
> + install_size = 0
> for po in pkglist:
> if po.state != TS_UPDATE and po.name not in installonlypkgs:
> + if po.state == TS_INSTALL:
Not sure where that leaves/should leave TS_TRUEINSTALL -- but I'm not
100% sure about the meanings of all transaction states, so not sure if
that needs to be honored here. If so, it should perhaps look like this:
+ if po.state in (TS_INSTALL, TS_TRUEINSTALL):
> + install_size += po.size
> continue
> local = po.localPkg()
> if os.path.exists(local):
> @@ -639,7 +642,7 @@ def predownload_hook(conduit):
> xpostdownload_hook(conduit)
> if more:
> conduit.info(2, "Package(s) data still to download: %s" %
> - format_number(more))
> + format_number(more + install_size))
>
> # Output stats. about delta downloads ... if we don't have deltas for
> # everything (pretty common), we want to output these before we start
--
Nils Philippsen "Those who would give up Essential Liberty to purchase
Red Hat a little Temporary Safety, deserve neither Liberty
[email protected] nor Safety." -- Benjamin Franklin, 1759
PGP fingerprint: C4A8 9474 5C4C ADE3 2B8F 656D 47D8 9B65 6951 3011
_______________________________________________
Yum-devel mailing list
[email protected]
http://lists.baseurl.org/mailman/listinfo/yum-devel
