GNU bug report logs - #55622
Bug in sort with keys and reverse, and version-sort and reverse

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Pádraig Brady <P <at> draigBrady.com>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#55622: closed (Bug in sort with keys and reverse, and
 version-sort and reverse)
Date: Wed, 25 May 2022 08:54:02 +0000

[Message part 1 (text/plain, inline)]

Your message dated Wed, 25 May 2022 09:53:08 +0100
with message-id <5180ddf7-dc90-1b15-db00-9a070f42cf1b <at> 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 <at> gnu.org.)


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

[Message part 2 (message/rfc822, inline)]

From: Larry Ploetz <lploetz <at> gmail.com>
To: bug-coreutils <at> gnu.org
Subject: Bug in sort with keys and reverse, and version-sort and reverse
Date: Tue, 24 May 2022 19:00:04 -0700

[Message part 3 (text/plain, inline)]

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

[Message part 4 (text/html, inline)]

[Message part 5 (message/rfc822, inline)]

From: Pádraig Brady <P <at> draigBrady.com>
To: Larry Ploetz <lploetz <at> gmail.com>, 55622-done <at> debbugs.gnu.org
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

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

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.