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

Re: [PATCH v2] completion: fix issue with process substitution not working on Git for Windows

From
JBJonas Berlin <jonas.berlin@outerspace.dyndns.org>
Date
Oct 27, 2011, 10:40 UTC
Message-ID
<20111027134043.56519ca4@outerspace.dyndns.org>
In-Reply-To
<20111027132754.1503b98b@outerspace.dyndns.org>

On Thu, 27 Oct 2011 13:27:54 +0300 Jonas Berlin <xkr47@outerspace.dyndns.org> wrote:

Show 13 quoted lines
> On Thu, 27 Oct 2011 11:05:30 +0200
> SZEDER Gábor <szeder@ira.uka.de> wrote:
> > The $output variable is not declared as local and therefore it leaks
> > into the environment.  But instead of declaring it local, why not
> > eliminate it altogether, and use the "$(git config ....)" command
> > substitution as here string?
> 
> Wouldn't this work:
> 
> 	git config -z --get-regexp '^(svn-remote\..*\.url|bash\.showupstream)$' 2>/dev/null | tr '\0\n' '\n ' | \
> 	while read key value; do
> 		...
> 	done
Sorry, please disregard, I didn't notice it was already dismissed in v1 of the PATCH..
- xkr47
Previous: Jonas Berlin
Message 10 of 10 in “completion: fix issue with process substitution not working on Git for Windows”
  1. completion: fix issue with process substitution not working on Git for WindowsStefan Naewe, Oct 25, 2011
  2. Johannes SixtOct 25, 2011
  3. Stefan NäweOct 26, 2011
  4. completion: fix issue with process substitution not working on Git for WindowsStefan Naewe, Oct 26, 2011
  5. Junio C HamanoOct 26, 2011
  6. Junio C HamanoOct 26, 2011
  7. Stefan NäweOct 27, 2011
  8. SZEDER GáborOct 27, 2011
  9. Jonas BerlinOct 27, 2011
  10. Jonas BerlinOct 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.