[Toybox] [PATCH] top: don't report GiB sizes in KiB.

[enh]

On Fri, Sep 11, 2020 at 4:48 PM Rob Landley <rob at landley.net> wrote:

> On 9/11/20 2:11 PM, enh wrote:
> >     Right now I have a patch to make the commas just care about
> "1,234.5" vs
> >     "1.234,5" as a binary choice (which covers the vast majority of the
> planet, and
> >     will never matter on android anyway because you nerfed that), vs a
> patch to
> >     revert the whole mess back to how it was before I started meddling
> with it.
> >
> >     (I looked at removing and commas but keeping the megabyte forcing,
> which gets
> >     back to "with 8 digits of space 64 gigs would still show as
> kilobytes, which is
> >     what you were objecting to in the first place, despite that being
> what debian
> >     does for me with 16 gigs of ram I.E. 8 digits"...)
> >
> > do you have an older or newer top than me? i gave my version number
> before
> > (procps-ng 3.3.16) but didn't explicitly say i was doing so in order for
> you to
> > be able to compare. i get MiB even on a box with only 16GiB.
>
> The top command won't tell me a versino, but dpkg says procps 3.3.12 but
> not -ng.
>

i think it will, it's just neither very helpful nor clear:

~$ top --version
top: inappropriate '-version'
Usage:
  top -hv | -bcEHiOSs1 -d secs -n max -u|U user -p pid(s) -o field -w [cols]
~$ top -v
  procps-ng 3.3.16
Usage:
  top -hv | -bcEHiOSs1 -d secs -n max -u|U user -p pid(s) -o field -w [cols]
~$

even when it works, it manages to make it look like a syntax error!

but, yeah, sounds like you have an older version, which at least answers
the question of which direction they're moving in.


> Rob
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.landley.net/pipermail/toybox-landley.net/attachments/20200914/82002849/attachment.html>