--- Begin Message ---
Subject: |
numfmt: invalid suffix 'k' |
Date: |
Fri, 12 Mar 2021 17:12:13 +0100 |
Hello,
the SI prefix for 'a thousand' is a lowercase k and not an uppercase K [1].
The default behavior of numfmt with '--from=si' option is therefore
contrary to expectation:
$ numfmt --from=si
500k # Should be accepted as valid SI
numfmt: invalid suffix in input: ‘500k’
$ numfmt --from=si
500K # Should result in error, 'K' stands for kelvin unit.
500000
$ numfmt --version
numfmt (GNU coreutils) 8.26
Packaged by Cygwin (8.26-2)
Copyright (C) 2016 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Written by Assaf Gordon.
Best regards,
Daniel.
[1] Page 143 of https://www.bipm.org/en/publications/si-brochure/
--- End Message ---
--- Begin Message ---
Subject: |
Re: bug#47103: numfmt: invalid suffix 'k' |
Date: |
Sun, 26 Nov 2023 16:59:36 +0000 |
User-agent: |
Mozilla Thunderbird |
On 26/11/2023 16:09, Sven Köhler wrote:
So Pádraig's patch does allow for parsing lowercase k, but it does not
change numfmt to use lowercase k in its output in si mode.
As Pádraig has shown, ls uses lowercase k in --si mode. So it uses
lowercase k for 1000. I think that numfmt should behave the same for
consistency reasons.
It does output lowercase 'k' in SI mode.
Attached is the full patch.
Marking this as done.
Will push this tomorrow.
thanks,
Pádraig
numfmt-k.patch
Description: Text Data
--- End Message ---