Zsh Mailing List Archive
Messages sorted by: Reverse Date, Date, Thread, Author

Re: A couple of queries

> In glob.c, in the function file_type() (which contains the delightful
> comment "screw POSIX"), there is commented-out code that would return a
> character '&' instead of '@' for a symbolic link that doesn't point
> anywhere accessible.  Considering that compatibility with "ls -F" isn't
> really an issue here (and '%' and '#' are returned for devices which ls
> doesn't do), why is this code not used?

I haven't seen that code.  I was going to remove that comment, but forgot.
The code has been posix-ified considerably since the start of the 2.6

> Secondly, I had an idea for a new option for a couple of builtins.
> compctl has an option -L that makes zsh output the actual compctl
> commands that one would have to input in order to get to the current
> set of completion options.  A similar option would be very useful for
> alias, bindkey (especially), limit, sched and typeset.  (typeset -L is
> already taken, so it would have to be something else, but the others
> can be -L.)  These options would not only be useful in normal use of
> the shell, but would also simplify the reporter script greatly (which
> btw won't run here -- awk syntax errors).  This option could be added
> to each of the builtins quite easily, and without (much) risk of
> breaking anything else.  What are people's opinions on this idea?
> Richard, would you be willing to add this to the baseline before the
> 2.7 release?

I'm not against adding them if they don't add too much code.  I think these
would be useful.  But I would prefer to use the same option for all of them.
So we need an option letter that is free on all of these builtins, and would
have to change compctl -L to match this.  Since compctl -L is new, this
shouldn't cause too much problem.  Also, in order to make things easier,
I would prefer that the changes to each builtin be put in a separate patch
if possible.

But I would prefer that we wait until the next beta release before this is
implemented.  My box is overflowing with patches.  I'm in the process
of making more changes to the code dealing with internal hash tables.  So
any patch adding these features would be broken by the time I receive it.
Also, with the big patches from Peter (for the history code) and some of the
other big patches, the code will change a lot in the next couple of releases.

Also, just to let everyone know, I'll probably also add the compctl -Q option
either in this release or the next one.

But, my current priority is still to continue the process of cleaning up the
code.  I want to fix the structural problems first, so things like Peter's
patch for history.c will go in first.  I'm also doing some work on param.c.

So the next beta release might take a while.  So what I'll probably do is
make code snapshots (test releases) available starting in about a week,
so people can help me work out all the bugs I'll probably be adding.


Messages sorted by: Reverse Date, Date, Thread, Author