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

Re: [PATCH 3/4] start_command: detect execvp failures early

From
Junio C Hamano <gitster@pobox.com>
Date
Jan 14, 2010, 21:31 UTC
Message-ID
<7viqb49xwb.fsf@alter.siamese.dyndns.org>
In-Reply-To
<201001101411.22418.j6t@kdbg.org>
Johannes Sixt <j6t@kdbg.org> writes:
Show 6 quoted lines
> Previously, failures during execvp could be detected only by
> finish_command. However, in some situations it is beneficial for the
> parent process to know earlier that the child process will not run.
>
> The idea to use a pipe to signal failures to the parent process and
> the test case were lifted from patches by Ilari Liusvaara.

I wonder if we can do this without pipe, perhaps using "vfork, exec, then update a variable"....

Previous: Johannes SixtNext: Johannes Sixt
Message 7 of 11 in “Imporove remote helper exec failure reporting”
  1. 0/2 Imporove remote helper exec failure reportingIlari Liusvaara, Jan 9, 2010
  2. 1/2 Report exec errors from run-commandIlari Liusvaara, Jan 9, 2010
  3. 0/4 Detect exec errors in start_command earlyJohannes Sixt, Jan 10, 2010
  4. 1/4 start_command: report child process setup errors to the parent's stderrJohannes Sixt, Jan 10, 2010
  5. 2/4 run-command: move wait_or_whine earlierJohannes Sixt, Jan 10, 2010
  6. 3/4 start_command: detect execvp failures earlyJohannes Sixt, Jan 10, 2010
  7. Junio C HamanoJan 14, 2010
  8. Johannes SixtJan 14, 2010
  9. Junio C HamanoJan 14, 2010
  10. 4/4 Improve error message when a transport helper was not foundJohannes Sixt, Jan 10, 2010
  11. 2/2 Improve transport helper exec failure reportingIlari Liusvaara, Jan 9, 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.