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

Re: [PATCH] run-command.c: need alloc.h for our own at-exit handler emulation

From
Junio C Hamano <gitster@pobox.com>
Date
May 16, 2023, 18:44 UTC
Message-ID
<xmqq353wune3.fsf@gitster.g>
In-Reply-To
<ZGPE9ozRChUJ3NGV@nand.local>
Taylor Blau <me@ttaylorr.com> writes:
Show 6 quoted lines
> OK, having now read both completely, I would say I have a vague
> preference for my version since it keeps the include at the top and
> unconditional, and has slightly more information in the patch message.
>
> But I do not prefer it so much over yours that I would be sad if you had
> already queued yours and didn't want to bother shuffling it around.

I do not have much preference between the two, either. Both lack one important description that we are reasonably confident that this breakage is limited to run-command.c and no other files.

Thanks.
Previous: Taylor BlauNext: Taylor Blau
Message 8 of 15 in “[BUG] Git 2.41.0-rc0 - Compile Error ALLOC_GROW”
  1. rsbecker@nexbridge.comMay 16, 2023
  2. Taylor BlauMay 16, 2023
  3. Junio C HamanoMay 16, 2023
  4. run-command.c: need alloc.h for our own at-exit handler emulationJunio C Hamano, May 16, 2023
  5. rsbecker@nexbridge.comMay 16, 2023
  6. Taylor BlauMay 16, 2023
  7. Taylor BlauMay 16, 2023
  8. Junio C HamanoMay 16, 2023
  9. Taylor BlauMay 16, 2023
  10. Junio C HamanoMay 16, 2023
  11. run-command.c: fix missing include under `NO_PTHREADS`Taylor Blau, May 16, 2023
  12. Junio C HamanoMay 16, 2023
  13. rsbecker@nexbridge.comMay 16, 2023
  14. Taylor BlauMay 16, 2023
  15. Elijah NewrenMay 18, 2023

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.