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

[RFC PATCH 0/2] protect git from a rogue editor

From
Carlo Marcelo Arenas Belón <carenas@gmail.com>
Date
Oct 2, 2021, 15:36 UTC
Message-ID
<20211002153654.52443-1-carenas@gmail.com>

The following series, expands git's terminal support to allow for saving/restoring its settings around an EDITOR call.

The rationale is better documented in multiple tickets for Windows users that had found themselves with a difficult to read output after doing a commit (either directly or while running a rebase), because they were using Windows Terminal and vi (the default).

It is also useful in POSIX systems which before this series will have to reset their terminals, if (for example) vi got killed while editing a commit message.

Posted as an RFC, because I suspect it might need additional changes to support Windows < 10 and because I wanted to add a patch that at least renames the "hconin" variable and maybe fix some spurious errors, or even add some logic to prevent the reset if the terminal wasn't messed with.

Carlo Marcelo Arenas Belón (2):
  terminal: teach git how to save/restore its terminal settings
  editor: save and reset terminal after calling EDITOR
 compat/terminal.c | 72 ++++++++++++++++++++++++++++++++++++++---------
 compat/terminal.h |  3 ++
 editor.c          |  4 +++
 3 files changed, 66 insertions(+), 13 deletions(-)
-- 
2.33.0.955.gee03ddbf0e
Next: Carlo Marcelo Arenas Belón
Message 1 of 14 in “protect git from a rogue editor”
  1. 0/2 protect git from a rogue editorCarlo Marcelo Arenas Belón, Oct 2, 2021
  2. 1/2 terminal: teach git how to save/restore its terminal settingsCarlo Marcelo Arenas Belón, Oct 2, 2021
  3. 2/2 editor: save and reset terminal after calling EDITORCarlo Marcelo Arenas Belón, Oct 2, 2021
  4. 0/2 protect git from a rogue editorCarlo Marcelo Arenas Belón, Oct 4, 2021
  5. 1/2 terminal: teach git how to save/restore its terminal settingsCarlo Marcelo Arenas Belón, Oct 4, 2021
  6. Junio C HamanoOct 4, 2021
  7. Carlo ArenasOct 4, 2021
  8. Junio C HamanoOct 4, 2021
  9. Carlo ArenasOct 4, 2021
  10. 2/2 editor: save and reset terminal after calling EDITORCarlo Marcelo Arenas Belón, Oct 4, 2021
  11. Junio C HamanoOct 4, 2021
  12. 0/2 protect git from a rogue editorCarlo Marcelo Arenas Belón, Oct 5, 2021
  13. 1/2 terminal: teach git how to save/restore its terminal settingsCarlo Marcelo Arenas Belón, Oct 5, 2021
  14. 2/2 editor: save and reset terminal after calling EDITORCarlo Marcelo Arenas Belón, Oct 5, 2021

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.