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

Re: [PATCH] add -i: ignore terminal escape sequences

From
Junio C Hamano <gitster@pobox.com>
Date
May 18, 2011, 03:52 UTC
Message-ID
<7vipt8fyxj.fsf@alter.siamese.dyndns.org>
In-Reply-To
<05ce7ccdb3f4e07724d430f6ea2a8c9730971c9d.1305645331.git.trast@student.ethz.ch>
<trast@student.ethz.ch> writes:
> I nearly managed to lose a bunch of uncommitted work today, but could
> salvage most of it from the pieces of diffs in the terminal
> scrollback.  Sigh.

I think the take-home lesson is that confirmation offered in the default mode is valuable. The "single-key" mode is another long rope that I would not use myself, but the users can choose to hang themselves with.

Jokes aside, it may make sense to offer an extra confirmation for "a" and possibly "s" in single-key mode. Unlike others, they are destructive when the changes you are splitting from the working tree is large-ish.

> Oh yeah, PS: I'm alive ;-)
Good to hear.
Previous: trast@student.ethz.chNext: Sverre Rabbelier
Message 2 of 6 in “add -i: ignore terminal escape sequences”
  1. add -i: ignore terminal escape sequencestrast@student.ethz.ch, May 17, 2011
  2. Junio C HamanoMay 18, 2011
  3. Sverre RabbelierMay 18, 2011
  4. Junio C HamanoMay 18, 2011
  5. Jeff KingMay 18, 2011
  6. Thomas RastMay 18, 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.