bug#55622: closed (Bug in sort with keys and reverse, and version-sort and reverse)

[GNU bug Tracking System]

Your message dated Wed, 25 May 2022 09:53:08 +0100
with message-id <5180ddf7-dc90-1b15-db00-9a070f42cf1b@draigBrady.com>
and subject line Re: bug#55622: Bug in sort with keys and reverse, and 
version-sort and reverse
has caused the debbugs.gnu.org bug report #55622,
regarding Bug in sort with keys and reverse, and version-sort and reverse
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
55622: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=55622
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems

--- Begin Message --- Subject: Bug in sort with keys and reverse, and version-sort and reverse Date: Tue, 24 May 2022 19:00:04 -0700 User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0) Gecko/20100101 Thunderbird/91.9.1

I think I've found a bug in sort (git branch master). The --reverse flag seems to be ignored when --keys are supplied.

larryp-MBP:bin larry$ ./sort --version
sort (GNU coreutils) 9.1.17-a351f
Copyright (C) 2022 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <https://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 Mike Haertel and Paul Eggert.
larryp-MBP:bin larry$ ./sort -r <<< $'a\nb'
b
a
larryp-MBP:bin larry$ ./sort -rk1,1 <<< $'a\nb'
a
b

This may be related to a bug reported in February where --version-sort sorted incorrectly.

larryp-MBP:bin larry$ ./sort -Vr <<< $'1\n2'
1
2
larryp-MBP:bin larry$ ./sort -V <<< $'1\n2'
1
2

Current version:

larryp-MBP:bin larry$ sort --version
sort (GNU coreutils) 9.1
Copyright (C) 2022 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <https://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 Mike Haertel and Paul Eggert.
larryp-MBP:bin larry$ sort -rk1,1 <<< $'a\nb'
b
a

Thanks,
· Larry

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#55622: Bug in sort with keys and reverse, and version-sort and reverse
>
>
>
>
> Date: 
>
> Wed, 25 May 2022 09:53:08 +0100
>
>
>
>
> User-agent: 
>
> Mozilla/5.0 (X11; Linux x86_64; rv:98.0) Gecko/20100101 Thunderbird/98.0
>
>
>
>
> On 25/05/2022 03:00, Larry Ploetz wrote:
> > I think I've found a bug in sort (git branch master). The --reverse flag
> > seems to be ignored when --keys are supplied.
> >
> >      larryp-MBP:bin larry$ ./sort --version
> >      sort (GNU coreutils) 9.1.17-a351f
> >      Copyright (C) 2022 Free Software Foundation, Inc.
> >      License GPLv3+: GNU GPL version 3 or 
> > later<https://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 Mike Haertel and Paul Eggert.
> >      larryp-MBP:bin larry$ ./sort -r <<< $'a\nb'
> >      b
> >      a
> >      larryp-MBP:bin larry$ ./sort -rk1,1 <<< $'a\nb'
> >      a
> >      b
>
> Thanks for the report.
> I pushed the following to address this
> https://git.sv.gnu.org/gitweb/?p=coreutils.git;a=commitdiff;h=4e0167ea1
>
> The existing tests are sufficient to catch this.
>
> thanks!
> Pádraig
>
>
>
> --- End Message ---