GNU bug report logs -
#5958
Sort-8.4 bug
Previous Next
Reported by: srodri <at> datsi.fi.upm.es
Date: Fri, 16 Apr 2010 12:14:02 UTC
Severity: normal
Merged with 5991
Fixed in version 8.5
Done: Eric Blake <eblake <at> redhat.com>
Bug is archived. No further changes may be made.
Full log
Message #29 received at 5958 <at> debbugs.gnu.org (full text, mbox):
On 20/04/10 09:37, Alan Curry wrote:
> A full investigation has revealed:
>
> This bug was introduced between coreutils 7.1 and 7.2, here:
>
>> commit 224a69b56b716f57e3a018af5a9b9379f32da3fc
>> Author: Pádraig Brady <P <at> draigBrady.com>
>> Date: Tue Feb 24 08:37:18 2009 +0000
>>
>> sort: Fix two bugs with determining the end of field
>>
>> * src/sort.c: When no specific number of chars to skip
>> is specified for the end field, always skip the whole field.
>> Also never include leading spaces from next field.
>> * tests/misc/sort: Add 2 new tests for these cases.
>> * NEWS: Mention this bug fix.
>> * THANKS: Add bug reporter.
>> Reported by Davide Canova.
>
> In the diff of that commit, an eword++ was removed from the case 'k' section
> of option parsing, where it did not affect traditional options, and added to
> the limfield() function, where it takes effect regardless of how fields were
> specified.
>
> So it fixed a -k option parsing bug and added a traditional option parsing
> bug.
Sigh. I didn't fully understand/consider the obsolete syntax when doing that.
I'll look at you patch tonight and push it in.
cheers,
Pádraig.
This bug report was last modified 15 years and 35 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.