GNU bug report logs - #7759
split(1) regression in v8.7-25-gbe10739

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 7759 in the body.
You can then email your comments to 7759 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: "Dmitry V. Levin" <ldv <at> altlinux.org>
To: bug-coreutils <at> gnu.org
Subject: split(1) regression in v8.7-25-gbe10739
Date: Thu, 30 Dec 2010 00:12:37 +0300

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

Hi,

I've got a bug report from Sergey Vlasov about split(1) regression
in coreutils v8.8: https://bugzilla.altlinux.org/24841

In short, the default suffix length is not 2 as it shall be according to POSIX.
Looks like the regression is due to commit v8.7-25-gbe10739.
For more details and the test case, please have a look at the bug report.


-- 
ldv

[Message part 2 (application/pgp-signature, inline)]

Message #8 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Pádraig Brady <P <at> draigBrady.com>
To: bug-coreutils <at> gnu.org
Subject: Re: bug#7759: split(1) regression in v8.7-25-gbe10739
Date: Thu, 30 Dec 2010 01:47:12 +0000

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

On 29/12/10 21:12, Dmitry V. Levin wrote:
> Hi,
> 
> I've got a bug report from Sergey Vlasov about split(1) regression
> in coreutils v8.8: https://bugzilla.altlinux.org/24841
> 
> In short, the default suffix length is not 2 as it shall be according to POSIX.
> Looks like the regression is due to commit v8.7-25-gbe10739.
> For more details and the test case, please have a look at the bug report.

Cringe :(

The frustrating this is that I noticed this
but dropped the fixup code when I was moving
local git branches around.

The attached should fix it.

I'll log a fedora rawhide bug also.

cheers,
Pádraig.

[split-suffix.diff (text/x-patch, attachment)]

Message #11 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: Pádraig Brady <P <at> draigBrady.com>
Cc: bug-coreutils <at> gnu.org
Subject: Re: bug#7759: split(1) regression in v8.7-25-gbe10739
Date: Thu, 30 Dec 2010 12:15:44 +0100

Pádraig Brady wrote:

> On 29/12/10 21:12, Dmitry V. Levin wrote:
>> Hi,
>>
>> I've got a bug report from Sergey Vlasov about split(1) regression
>> in coreutils v8.8: https://bugzilla.altlinux.org/24841
>>
>> In short, the default suffix length is not 2 as it shall be according to POSIX.
>> Looks like the regression is due to commit v8.7-25-gbe10739.
>> For more details and the test case, please have a look at the bug report.
>
> Cringe :(
>
> The frustrating this is that I noticed this
> but dropped the fixup code when I was moving
> local git branches around.
>
> The attached should fix it.
...
> Subject: [PATCH] split: fix the suffix length calculation
>
> * src/split.c (set_suffix_length): Only auto calculate
> the suffix length when the number of files is specified.
> * tests/misc/split-a: Add a case to trigger the bug.
> * NEWS: Mention the fix.

Hi Pádraig
Thanks for the quick fix.
Please include the bugzilla URL and credit Sergey and Dmitry
in the commit log.  It'd be good to list Sergey's
name/email in THANKS, too.

Message #16 received at 7759-done <at> debbugs.gnu.org (full text, mbox):

From: Pádraig Brady <P <at> draigBrady.com>
To: 7759-done <at> debbugs.gnu.org
Subject: Re: bug#7759: split(1) regression in v8.7-25-gbe10739
Date: Thu, 30 Dec 2010 16:32:05 +0000

On 30/12/10 11:15, Jim Meyering wrote:
> 
> Hi Pádraig
> Thanks for the quick fix.
> Please include the bugzilla URL and credit Sergey and Dmitry
> in the commit log.  It'd be good to list Sergey's
> name/email in THANKS, too.

Done and pushed.

thanks,
Pádraig.

Previous Next

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