bug#12260: rm -d in coreutils 8.19

2012-08-23 Thread Rob Day
On 22 August 2012 23:23, Robert Day wrote: > I've attached a patch which fixes this bug and adds a test of that code > path. The fixes can also be retrieved from > https://github.com/rkd91/coreutils_rm_di_patch. > I've made a couple of related fixes (comments/code niceness and adding a NEWS item

bug#12260: [patch] rm -d in coreutils 8.19

2012-08-23 Thread Jim Meyering
Robert Day wrote: > Looks good to me; feel free to push it. I'll be mindful of your comments on > else-statement style, comment formatting etc. if and when I submit another > coreutils patch. Pushed. Looking forward to more from you. BTW, I noticed that your message was more than double the leng

bug#12260: [patch] rm -d in coreutils 8.19

2012-08-23 Thread Robert Day
Looks good to me; feel free to push it. I'll be mindful of your comments on else-statement style, comment formatting etc. if and when I submit another coreutils patch. Cheers, Rob On 23 August 2012 19:53, Jim Meyering wrote: > Jim Meyering wrote: > > Jim Meyering wrote: > >> Pádraig Brady wrote

bug#12260: [patch] rm -d in coreutils 8.19

2012-08-23 Thread Jim Meyering
Jim Meyering wrote: > Jim Meyering wrote: >> Pádraig Brady wrote: >>> On 08/23/2012 12:17 PM, Robert Day wrote: On 22 August 2012 23:23, Robert Day wrote: > I've attached a patch which fixes this bug and adds a test of that code > path. The fixes can also be retrieved from >

bug#12260: [patch] rm -d in coreutils 8.19

2012-08-23 Thread Jim Meyering
Jim Meyering wrote: > Pádraig Brady wrote: >> On 08/23/2012 12:17 PM, Robert Day wrote: >>> On 22 August 2012 23:23, Robert Day wrote: >>> I've attached a patch which fixes this bug and adds a test of that code path. The fixes can also be retrieved from https://github.com/rkd91/cor

bug#12260: [patch] rm -d in coreutils 8.19

2012-08-23 Thread Jim Meyering
Pádraig Brady wrote: > On 08/23/2012 12:17 PM, Robert Day wrote: >> On 22 August 2012 23:23, Robert Day wrote: >> >>> I've attached a patch which fixes this bug and adds a test of that code >>> path. The fixes can also be retrieved from >>> https://github.com/rkd91/coreutils_rm_di_patch. >>> >> >>

bug#12260: [patch] rm -d in coreutils 8.19

2012-08-23 Thread Pádraig Brady
On 08/23/2012 12:17 PM, Robert Day wrote: > On 22 August 2012 23:23, Robert Day wrote: > >> I've attached a patch which fixes this bug and adds a test of that code >> path. The fixes can also be retrieved from >> https://github.com/rkd91/coreutils_rm_di_patch. >> > > I've made a couple of relate

bug#12260: [patch] rm -d in coreutils 8.19

2012-08-23 Thread Robert Day
On 22 August 2012 23:23, Robert Day wrote: > I've attached a patch which fixes this bug and adds a test of that code > path. The fixes can also be retrieved from > https://github.com/rkd91/coreutils_rm_di_patch. > I've made a couple of related fixes (comments/code niceness and adding a NEWS item

bug#12264: join command

2012-08-23 Thread Pádraig Brady
forcemerge 12264 6366 stop On 08/23/2012 07:08 AM, Szilard Albert wrote: > Hi, > > I am trying to use the join command to join the below files, and it only > works for up to the pair of 1000. > Any suggestions? join(1) doesn't support numeric comparison currently. It should, and that's already t

bug#12264: join command

2012-08-23 Thread Szilard Albert
Hi, I am trying to use the join command to join the below files, and it only works for up to the pair of 1000. Any suggestions? cheers Sid 900-164.06 901-164.05 902-164.03 903-164.02 904-164.01 905-163.99 906-163.98