Andreas Schwab <[EMAIL PROTECTED]> wrote:
> Jim Meyering <[EMAIL PROTECTED]> writes:
>> Andreas Schwab <[EMAIL PROTECTED]> wrote:
>>> Bruce Korb <[EMAIL PROTECTED]> writes:
>>>
>>>> Yep. That's the reason for the "compare-" prefix. I didn't like
>>>> ``--compare-version-sort'' for some sort of reason, too. Ultimately,
>>>> someone pick another name if "compare-version" is aesthetically bad.
>>>
>>> How about making it --sort=KEYWORD, with KEYWORD in
>>> {general-numeric,month,numeric,random,version}?
>>
>> I like it.
>
> Here's a patch to add --sort.
>
> Andreas.
>
> 2008-02-09 Andreas Schwab <[EMAIL PROTECTED]>
>
> Add --sort option.
> * src/sort.c (SORT_OPTION): New enum.
> (sort_args, sort_types): Define.
> (usage, long_options, main): New option --sort.
Thanks for writing that.
I appreciate your including a ChangeLog entry.
Do you feel like adding a few tests (tests/sort/Test.pm) to exercise
the new feature, and adding to coreutils.texi and NEWS?
_______________________________________________
Bug-coreutils mailing list
Bug-coreutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-coreutils
