Re: [PATCH for maint] git-completion: fix zsh support

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

 



Felipe Contreras wrote:

> No, the scope remains local.

Is this local in the sense of typeset used in ksh functions declared
as "function f" or local in the sense of typeset used in ksh functions
declared as "f ()"?  If the latter, I think you misunderstood me --- I
never meant to suggest otherwise.

> In fact, if you follow the link I posted, that's precisely the fix the
> zsh guys were pushing for. And the it is already merged:
> http://zsh.git.sourceforge.net/git/gitweb.cgi?p=zsh/zsh;a=commitdiff;h=e880604f029088f32fb1ecc39213d720ae526aaa

Now this changes things (since it amounts to a guarantee that the
bash completion emulation has already extracted all it needs from
$words before calling a completion function).  What happens if someone
calls "compgen -F" after words is hidden?

> Well, it's hosted on debian.org, and I haven't seen it used anywhere
> else. I just don't know how else to identify that project.

Here you go: http://pkgs.fedoraproject.org/gitweb/?p=bash-completion.git;a=tree

It's called bash-completion or bash_completion.  It really has
basically nothing to do with Debian; they just are using Debian
infrastructure.  But hey, call it what you want.  I don't like
arguing.

> I think the simplest fix is the one I'm proposing

I'm happy with it as long as the zsh people are committed to making
sure it continues to work (which it sounds like they are, luckily).
I suppose it is intended to obsolete the third patch from GÃbor's
series?  Could you provide an explanation for the commit log,
something to the effect that

 - the words array has special meaning
 - that produces such-and-such puzzling symptoms
 - zsh 4.3.12 (?) will fix it by using "typeset -h" to hide it when
   running completion functions designed for bash
 - we can make the same fix to work correctly with earlier versions of
   zsh, and that's what this patch does

?

Sorry I have been so dense --- hopefully I understand it roughly
now.

Regards,
Jonathan
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]