Re: bug in ksh tab complete

2013-06-04 Thread LEVAI Daniel
On h, jún 03, 2013 at 11:05:11 -0400, Ted Unangst wrote: [...] > > Didn't send the diff; I think because of the general lack of interest in > > ksh patches in the past. > > I don't think that's always true, sometimes the interested people > aren't interested that day, or in that patch. But as a pr

Re: bug in ksh tab complete

2013-06-03 Thread Jérémie Courrèges-Anglas
Philip Guenther writes: > On Mon, Jun 3, 2013 at 9:29 AM, Jérémie Courrèges-Anglas > wrote: >> Ted Unangst writes: >>> If ksh is going to treat : as magic, then it needs to escape it when >>> autocompleting. (step 2 above) >> >> I do agree, but... why should ':' be special? > > So that things

Re: bug in ksh tab complete

2013-06-03 Thread Marc Espie
On Mon, Jun 03, 2013 at 09:33:25AM -0700, Philip Guenther wrote: > On Mon, Jun 3, 2013 at 9:29 AM, Jérémie Courrèges-Anglas > wrote: > > Ted Unangst writes: > >> If ksh is going to treat : as magic, then it needs to escape it when > >> autocompleting. (step 2 above) > > > > I do agree, but... wh

Re: bug in ksh tab complete

2013-06-03 Thread Philip Guenther
On Mon, Jun 3, 2013 at 9:29 AM, Jérémie Courrèges-Anglas wrote: > Ted Unangst writes: >> If ksh is going to treat : as magic, then it needs to escape it when >> autocompleting. (step 2 above) > > I do agree, but... why should ':' be special? So that things like PATH=/usr/local/bin:/usr/b and

Re: bug in ksh tab complete

2013-06-03 Thread Jérémie Courrèges-Anglas
Ted Unangst writes: [...] > If ksh is going to treat : as magic, then it needs to escape it when > autocompleting. (step 2 above) I do agree, but... why should ':' be special? -- Jérémie Courrèges-Anglas PGP Key fingerprint: 61DB D9A0 00A4 67CF 2A90 8961 6191 8FBF 06A1 1494

Re: bug in ksh tab complete

2013-06-03 Thread Ted Unangst
On Mon, Jun 03, 2013 at 09:45, LEVAI Daniel wrote: > --- edit.c.orig 2012-10-31 19:21:31.742319303 +0100 > +++ edit.c2012-10-31 19:21:44.031181937 +0100 > @@ -809,7 +809,7 @@ > int rval = 0; > > for (add = 0, wlen = len; wlen - add > 0; add++) { > - if (strchr("\"#$&'()*;<=>?

Re: bug in ksh tab complete

2013-06-03 Thread LEVAI Daniel
On v, jún 02, 2013 at 20:02:17 -0400, Ted Unangst wrote: > (1) I'm in src/usr.sbin/pkg_add. I type "vi pod/". ksh prints some > completions for me: > > athens:~/src/usr.sbin/pkg_add> vi pod/ > CVS/ OpenBSD::PackingElement.pod ... > > (2) I type "Open". ksh completes a little more for me: > at