git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] completion: move private shopt shim for zsh to __git_ namespace

From
Jonathan Nieder <jrnieder@gmail.com>
Date
Apr 27, 2011, 23:00 UTC
Message-ID
<20110427230008.GD18596@elie>
In-Reply-To
<BANLkTinKgubAeFy+mJ=hGdG8c-QXjyNTcA@mail.gmail.com>
Felipe Contreras wrote:
> I have been looking for a way to have local aliases or functions, but
> I can't find any.
It is possible to do
	if [[ -n ${ZSH_VERSION-} ]]; then
		alias __git_local="typeset -h"
	else
		alias __git_local=local
	fi
but let's consider that for a moment.
 1. It's ugly (it means completion code would use a dialect where the
    ordinary "local" keyword has to be spelled differently).
 2. It's ugly (use of aliases in scripts sets off alarm bells.  As
    Almquist's sh manual says:
	Aliases provide a convenient way for naïve users to create
	shorthands for commands without having to learn how to create
	functions with arguments.  They can also be used to create
	lexically obscure code.  This use is discouraged.)
 3. It's fragile (maybe some day a function from zsh's completion
    library that we call will look at $words and get utterly confused).

I don't mean to sound so negative; actually I am very happy to see us getting closer to a full understanding of the problem and relevant constraints.

Previous: Felipe ContrerasNext: Jonathan Nieder
Message 13 of 28 in “git-completion: fix zsh support”
  1. git-completion: fix zsh supportFelipe Contreras, Apr 27, 2011
  2. Jonathan NiederApr 27, 2011
  3. Felipe ContrerasApr 27, 2011
  4. Junio C HamanoApr 27, 2011
  5. completion: avoid "words" as variable name for zsh portabilityJonathan Nieder, Apr 27, 2011
  6. Felipe ContrerasApr 27, 2011
  7. Jonathan NiederApr 27, 2011
  8. Felipe ContrerasApr 27, 2011
  9. John SzakmeisterApr 27, 2011
  10. Felipe ContrerasApr 27, 2011
  11. completion: move private shopt shim for zsh to __git_ namespaceJonathan Nieder, Apr 27, 2011
  12. Felipe ContrerasApr 27, 2011
  13. Jonathan NiederApr 27, 2011
  14. Jonathan NiederMay 6, 2011
  15. Felipe ContrerasMay 6, 2011
  16. SZEDER GáborMay 8, 2011
  17. SZEDER GáborApr 28, 2011
  18. 1/3 bash: don't modify the $cur variable in completion functionsSZEDER Gábor, Apr 28, 2011
  19. 2/3 bash: remove unnecessary _get_comp_words_by_ref() invocationsSZEDER Gábor, Apr 28, 2011
  20. 3/3 bash: don't declare 'local words' to make zsh happySZEDER Gábor, Apr 28, 2011
  21. Felipe ContrerasMay 3, 2011
  22. Felipe ContrerasApr 28, 2011
  23. Junio C HamanoApr 28, 2011
  24. Felipe ContrerasApr 28, 2011
  25. Felipe ContrerasApr 27, 2011
  26. Junio C HamanoApr 27, 2011
  27. Felipe ContrerasApr 27, 2011
  28. Jonathan NiederApr 27, 2011

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.