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

Re: [PATCH v2] builtin-tag.c: allow arguments in $EDITOR

From
LRLuciano Rocha <luciano@eurotux.com>
Date
Dec 20, 2007, 12:06 UTC
Message-ID
<20071220120601.GA15290@bit.office.eurotux.com>
In-Reply-To
<Pine.LNX.4.64.0712201255510.14355@wbgn129.biozentrum.uni-wuerzburg.de>
On Thu, Dec 20, 2007 at 12:58:59PM +0100, Johannes Schindelin wrote:
Show 14 quoted lines
> Hi,
> 
> On Thu, 20 Dec 2007, Luciano Rocha wrote:
> 
> > The previous sh version of git-commit evaluated the value of the defined 
> > editor, thus allowing arguments.
> > 
> > Make the builtin version work the same, by adding an explicit check for 
> > arguments in the editor command, and extract them to an additional 
> > argument.
> 
> Anything wrong with that patch?
> 
> http://article.gmane.org/gmane.comp.version-control.git/68444

No, I just missed it in the mailing list. That patch also supports any number of whitespace/arguments.

-- 
Luciano Rocha <luciano@eurotux.com>
Eurotux Informática, S.A. <http://www.eurotux.com/>
Previous: Johannes SchindelinNext: Junio C Hamano
Message 4 of 7 in “builtin-tag.c: allow arguments in $EDITOR”
  1. builtin-tag.c: allow arguments in $EDITORLuciano Rocha, Dec 19, 2007
  2. builtin-tag.c: allow arguments in $EDITORLuciano Rocha, Dec 20, 2007
  3. Johannes SchindelinDec 20, 2007
  4. Luciano RochaDec 20, 2007
  5. Junio C HamanoDec 20, 2007
  6. Steven GrimmDec 21, 2007
  7. Johannes SchindelinDec 22, 2007

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.