Gentoo Archives: gentoo-portage-dev

From: Zac Medico <zmedico@g.o>
To: gentoo-portage-dev@l.g.o, Fabian Groffen <grobian@g.o>
Subject: Re: [gentoo-portage-dev] [PATCH v2] __dyn_install: improve reporting of build and image sizes
Date: Sun, 27 Aug 2017 19:53:25
Message-Id: e0d8503b-c874-005f-4074-e19dd657647b@gentoo.org
In Reply to: [gentoo-portage-dev] [PATCH v2] __dyn_install: improve reporting of build and image sizes by Fabian Groffen
1 On 08/27/2017 08:06 AM, Fabian Groffen wrote:
2 > Prior to this commit, the reported sizes would look like:
3 >
4 > * Final size of build directory: 34942 KiB
5 > * Final size of installed tree: 5627 KiB
6 >
7 > Because the sizes aren't aligned, it is hard to (visually) compare them.
8 > On top of this, because the numbers are sometimes bigger, print a human
9 > friendly size after the KiB size if applicable, like so:
10 >
11 > * Final size of build directory: 1906 KiB (1.8 MiB)
12 > * Final size of installed tree: 7 KiB
13 >
14 > It should be noted that in case both sizes have a human-readable
15 > variant, they are also aligned.
16 >
17 > The helper functions are defined and used in a subshell to avoid
18 > pollution of the caller's environment.
19 > ---
20 > bin/phase-functions.sh | 53 ++++++++++++++++++++++++++++++++++++++++++++++----
21 > 1 file changed, 49 insertions(+), 4 deletions(-)
22
23 Looks good. Please merge.
24 --
25 Thanks,
26 Zac

Replies