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

Re: [PATCH] send-email: lazily assign editor variable

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 26, 2010, 19:32 UTC
Message-ID
<7v39zmlvz1.fsf@alter.siamese.dyndns.org>
In-Reply-To
<20100325051743.GB5784@progeny.tock>
Jonathan Nieder <jrnieder@gmail.com> writes:
Show 6 quoted lines
> Yes, please.  It would be more precise to
>
>   s/without a tty/with TERM=dumb/
>
> in the second paragraph but regardless this is a good description of the
> problem.

Strictly speaking, with TERM=dumb the true "vi" fell back to the ex mode and should have been usable. A purist might say it is a bug that we fatal out in this case (I know that it is deliberately done to help newbies from common confusion by not running any editor with TERM=dumb, but to a purist in me it feels somewhat wrong).

Previous: Jonathan NiederNext: Jonathan Nieder
Message 8 of 14 in “[REGRESSION] git var GIT_EDITOR fails without tty”
  1. Uwe Kleine-KönigMar 22, 2010
  2. send-email: lazily assign editor variableMichael J Gruber, Mar 22, 2010
  3. Uwe Kleine-KönigMar 22, 2010
  4. Jonathan NiederMar 23, 2010
  5. Michael J GruberMar 23, 2010
  6. Junio C HamanoMar 24, 2010
  7. Jonathan NiederMar 25, 2010
  8. Junio C HamanoMar 26, 2010
  9. Jonathan NiederMar 26, 2010
  10. Michael J GruberMar 25, 2010
  11. Junio C HamanoMar 26, 2010
  12. send-email: do not check for editor until neededJonathan Nieder, Mar 22, 2010
  13. Peter KjellerstedtMar 23, 2010
  14. Jonathan NiederMar 23, 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.