zsh-workers
 help / color / mirror / code / Atom feed
* bug in completion for "rm" in 4.3.9
@ 2008-12-21 22:03 Greg Klanderman
  2008-12-22  5:56 ` Bart Schaefer
  0 siblings, 1 reply; 3+ messages in thread
From: Greg Klanderman @ 2008-12-21 22:03 UTC (permalink / raw)
  To: Zsh list


Hi,

The completion for "rm" has a minor bug in that if the string you are
trying to complete is a valid completion and also the prefix of other
completions, it is ignored:

[~] phl| mkdir foo
[~] phl| cd foo
[foo] phl| touch foo foobar
[foo] phl| rm f    <list-choices> => "foo", "foobar"
[foo] phl| rm foo  <list-choices> => "foobar"

by comparison, completions for "ls" do not have this problem:

[foo] phl| ls f    <list-choices> => "foo", "foobar"
[foo] phl| ls foo  <list-choices> => "foo", "foobar"

The problem seems to be in this line:

|    ignored=(${line//(#m)[\[\]()\\*?#<>~\^]/\\$MATCH})

but I don't really understand what it's doing so cannot suggest
the right fix.

thanks,
Greg


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: bug in completion for "rm" in 4.3.9
  2008-12-21 22:03 bug in completion for "rm" in 4.3.9 Greg Klanderman
@ 2008-12-22  5:56 ` Bart Schaefer
  2008-12-22 16:04   ` Greg Klanderman
  0 siblings, 1 reply; 3+ messages in thread
From: Bart Schaefer @ 2008-12-22  5:56 UTC (permalink / raw)
  To: Zsh list

On Dec 21,  5:03pm, Greg Klanderman wrote:
} 
} The completion for "rm" has a minor bug

http://www.zsh.org/mla/users/2008/msg01093.html


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: bug in completion for "rm" in 4.3.9
  2008-12-22  5:56 ` Bart Schaefer
@ 2008-12-22 16:04   ` Greg Klanderman
  0 siblings, 0 replies; 3+ messages in thread
From: Greg Klanderman @ 2008-12-22 16:04 UTC (permalink / raw)
  To: zsh-workers

>>>>> Bart Schaefer <schaefer@brasslantern.com> writes:

> http://www.zsh.org/mla/users/2008/msg01093.html

Doh, sorry about that.  I was looking at a CVS checkout and saw that
commit but running 4.3.9 and thinking that change was incorporated.

Any idea when 4.3.10 will be released?

Is there a CVS commits mailing list?

greg


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-12-22 16:04 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-12-21 22:03 bug in completion for "rm" in 4.3.9 Greg Klanderman
2008-12-22  5:56 ` Bart Schaefer
2008-12-22 16:04   ` Greg Klanderman

Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).