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

Re: Apology/Bug report: git-send-email sends everything on Ctrl+C

From
Will Palmer <wmpalmer@gmail.com>
Date
May 26, 2010, 08:05 UTC
Message-ID
<1274861112.2074.2.camel@wpalmer.simply-domain>
In-Reply-To
<4BFCC4B5.6070408@vilain.net>
On Wed, 2010-05-26 at 18:50 +1200, Sam Vilain wrote:
Show 16 quoted lines
> Sam Vilain wrote:
> > Expect a mail storm shortly, sorry folks.
> >   
> 
> I just had a look at patching git-send-email.perl; but it looks like it
> is checking $? correctly at least since beece9da;
> 
>             system('sh', '-c', $editor.' "$@"', $editor, $_);
>             if (($? & 127) || ($? >> 8)) {
>                 die("the editor exited uncleanly, aborting everything");
>             }
> 
> I think I must have actually hit Ctrl-Z, not Ctrl-C.  So, it's all my
> fault and I apologize profusely.
> 
> Sam

I'd consider anything that can make "accidentally sending 100+ patches" possible to be a bug. How does "it was Ctrl-Z, not Ctrl-C" make a difference here?

Previous: Sam VilainNext: Sam Vilain
Message 3 of 8 in “Apology/Bug report: git-send-email sends everything on Ctrl+C”
  1. Sam VilainMay 26, 2010
  2. Sam VilainMay 26, 2010
  3. Will PalmerMay 26, 2010
  4. Sam VilainMay 26, 2010
  5. Sverre RabbelierMay 26, 2010
  6. Michael J GruberMay 26, 2010
  7. Sam VilainMay 26, 2010
  8. Michael J GruberMay 26, 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.