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 |