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

Re: [PATCH] completion: fix zsh check under bash with 'set -u'

From
SZEDER Gábor <szeder@ira.uka.de>
Date
Oct 28, 2010, 13:51 UTC
Message-ID
<20101028135154.GC1877@neumann>
In-Reply-To
<1288228101-23000-1-git-send-email-lodatom@gmail.com>
Hi Mark,
On Wed, Oct 27, 2010 at 09:08:21PM -0400, Mark Lodato wrote:
Show 9 quoted lines
> Commit 06f44c3 (completion: make compatible with zsh) broke bash
> compatibility with 'set -u': a warning was generated when checking
> $ZSH_VERSION.  The solution is to supply a default value, using
> ${ZSH_VERSION-}.  Thanks to SZEDER Gábor for the fix.
> 
> Signed-off-by: Mark Lodato <lodatom@gmail.com>
> ---
>  contrib/completion/git-completion.bash |    6 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)

Thanks. It works with bash and set -u, so if it works with zsh as well, then I'm for it.

While you are around, could you please have a look at this thread?
http://thread.gmane.org/gmane.comp.version-control.git/160103/focus=160135

Thanks, Gábor

Previous: Mark Lodato
Message 2 of 2 in “completion: fix zsh check under bash with 'set -u'”
  1. completion: fix zsh check under bash with 'set -u'Mark Lodato, Oct 28, 2010
  2. SZEDER GáborOct 28, 2010

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.