[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: fix thousands grouping issues
From: |
Jim Meyering |
Subject: |
Re: fix thousands grouping issues |
Date: |
Fri, 29 Dec 2023 21:47:14 -0800 |
On Wed, Dec 27, 2023 at 4:14 PM Pádraig Brady <P@draigbrady.com> wrote:
> On 27/12/2023 20:52, Jim Meyering wrote:
> > Thanks. Nice improvement.
> > I compared --help and --version output, which showed no difference
> > other than the intended ones.
>
> Thanks for the review. Pushed.
>
> >
> > make check passed, modulo this unrelated failure on Fedora 38:
> >
> > numfmt.pl: test lcl-dbl-to-human-1: stdout mismatch, comparing
> > lcl-dbl-to-human-1.1 (expected) and lcl-dbl-to-human-1.O (actual)
> > *** lcl-dbl-to-human-1.1 Wed Dec 27 12:47:48 2023
> > --- lcl-dbl-to-human-1.O Wed Dec 27 12:47:48 2023
> > ***************
> > *** 1 ****
> > ! 1,1K
> > --- 1 ----
> > ! 1,1k
>
> Oh interesting.
> I suspect that's due to glibc-all-langpacks not being installed on your
> system.
> That would mean a regular space was used as the thousands grouping char,
> and thus the test was run for you, and not for me.
>
> I see a similar skipping issue in tests/sort/sort-h-thousands-sep.sh,
> and in fact that was hiding a sort bug introduced in v9.1 !
>
> Hopefully the 3 patches attached addresses these issues.
> Paul are you Ok with the change to sort in the last patch?
Thanks. That looks right to me.
The other two patches look impeccable, too.
- Re: [PATCH] chgrp: --from parameter similar to chown, Pádraig Brady, 2023/12/15
- Re: [PATCH] chgrp: --from parameter similar to chown, Bernhard Voelker, 2023/12/16
- Re: [PATCH] chgrp: --from parameter similar to chown, Pádraig Brady, 2023/12/17
- Re: [PATCH] chgrp: --from parameter similar to chown, Pádraig Brady, 2023/12/27
- Re: [PATCH] chgrp: --from parameter similar to chown, Jim Meyering, 2023/12/27
- Re: [PATCH] chgrp: --from parameter similar to chown, Pádraig Brady, 2023/12/27
- Re: [PATCH] chgrp: --from parameter similar to chown, Pádraig Brady, 2023/12/27
- Re: [PATCH] chgrp: --from parameter similar to chown, Jim Meyering, 2023/12/27
- fix thousands grouping issues, Pádraig Brady, 2023/12/27
- Re: fix thousands grouping issues,
Jim Meyering <=
- Re: fix thousands grouping issues, Pádraig Brady, 2023/12/30