On 12/05/10 14:55, Eric Blake wrote:
> On 05/12/2010 07:53 AM, Eric Blake wrote:
>> On 05/11/2010 05:39 PM, Pádraig Brady wrote:
>>> The attached patch gives warnings about questionable
>>> option combinations. For example:
>>>
>>> $ sort --debug -rb -k1,1n /dev/null
>>> ! options `-b' are ignored
On 05/12/2010 07:53 AM, Eric Blake wrote:
> On 05/11/2010 05:39 PM, Pádraig Brady wrote:
>> The attached patch gives warnings about questionable
>> option combinations. For example:
>>
>> $ sort --debug -rb -k1,1n /dev/null
>> ! options `-b' are ignored
>> ! option `-r' only applies to last-resort
On 12/05/10 12:08, Jim Meyering wrote:
> Pádraig Brady wrote:
>> The attached patch allows one to:
>>
>> $ printf "one 2 3e3e" | .sort --debug -k3,3g -k2,2
>> one 2 3e3e
>> ___
>>__
>> __
>
> Nice. Thanks for writing all of that.
> These changes are sure to be useful.
>
> I hav
On 05/11/2010 05:39 PM, Pádraig Brady wrote:
> The attached patch gives warnings about questionable
> option combinations. For example:
>
> $ sort --debug -rb -k1,1n /dev/null
> ! options `-b' are ignored
> ! option `-r' only applies to last-resort comparison
That looks awkward, both when compare
Pádraig Brady wrote:
> The attached patch allows one to:
>
> $ printf "one 2 3e3e" | .sort --debug -k3,3g -k2,2
> one 2 3e3e
> ___
>__
> __
Nice. Thanks for writing all of that.
These changes are sure to be useful.
I have merely glanced through them (look fine so far),
but woul
On 12/05/10 00:39, Pádraig Brady wrote:
> The attached patch gives warnings about questionable
> option combinations. For example:
>
> $ sort --debug -rb -k1,1n /dev/null
> ! options `-b' are ignored
> ! option `-r' only applies to last-resort comparison
Oops, The previous patch warned about -r e
jeff.liu wrote:
> Hello All,
...
> Would you guys please review and consider apply the patches if there is no
> other issue?
Thanks for yet another iteration, and sorry it's taking so long.
>>From f8c78794a70f1fb45a2c61c8bbeca344087287ab Mon Sep 17 00:00:00 2001
> From: Jie Liu
> Date: Fri, 7 M
