bug-coreutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: sort -o x -o y


From: Jim Meyering
Subject: Re: sort -o x -o y
Date: Fri, 05 Sep 2003 00:14:17 +0200

Paul Eggert <address@hidden> wrote:
> Dan Jacobson <address@hidden> writes:
>
>> $ echo a|sort -o x -o y
>> $ ls
>> y
>
> POSIX allows this behavior, but it's admittedly weird.
>
> I think that option order should not matter, unless POSIX or the
> documentation explicitly says otherwise.  So I propose the following
> patch.  While looking into this problem I noticed that sort's -t
> option doesn't let you specify a NUL as a field separator (this is a
> related issue since 'sort' uses 0 to represent "no option specified
> yet").  Also, the documentation and usage strings incorrectly say
> "white space" several places where they should say "blanks".  Here's
> a patch for these problems.
>
> 2003-09-02  Paul Eggert  <address@hidden>
>
>       * NEWS: sort -t '\0' now uses a NUL tab.
>       sort option order no longer matters, unless POSIX requires it.
>       * doc/coreutils.texi (sort invocation): -d now overrides -i.
>       "whitespace" -> "blanks"; "whitespace" isn't correct.
>       -t '\0' now specifies a NUL tab.
>       * src/sort.c (usage): Say "blanks" instead of "whitespace",
>       Similar fixes for many comments.
>       (TAB_DEFAULT): New constant, so that we can support NUL as
>       the field separator.
>       (tab): Now int, not char.  Initialize to TAB_DEFAULT.
>       (specify_sort_size): If multiple sizes are specified, use the largest.
>       (begfield, limfield): Support NUL tab char.
>       (set_ordering): Do not let -i override -d.
>       (main): Report an error if incompatible -o or -t options are given.
>       Report an error for "-t ''".  Allow "-t '\0'" to specify a NUL tab.

Thank you both!
I've applied that patch and added a couple of tests.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]