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

Re: [PATCH 4/7] run_command: report system call errors instead of returning error codes

From
Junio C Hamano <gitster@pobox.com>
Date
Jul 5, 2009, 19:14 UTC
Message-ID
<7v3a9bj61m.fsf@alter.siamese.dyndns.org>
In-Reply-To
<200907052101.13741.j6t@kdbg.org>
Johannes Sixt <j6t@kdbg.org> writes:
Show 6 quoted lines
> On Samstag, 4. Juli 2009, Johannes Sixt wrote:
>> +	if (status >= 0 || errno != ENOENT)
>> +		exit(status & 0xff);
>
> With my replacement patch 1/7 this '& 0xff' is no longer necessary. Would you
> kindly squash this in?
Sounds sensible.  Thanks.
Previous: Johannes SixtNext: Johannes Sixt
Message 10 of 12 in “rework run_command error reporting”
  1. 0/7 rework run_command error reportingJohannes Sixt, Jul 4, 2009
  2. 1/7 Truncate result of run_command that is used in exit() to lowest 8 bitsJohannes Sixt, Jul 4, 2009
  3. 2/7 MinGW: simplify waitpid() emulation macrosJohannes Sixt, Jul 4, 2009
  4. 3/7 run_command: return exit code as positive valueJohannes Sixt, Jul 4, 2009
  5. 4/7 run_command: report system call errors instead of returning error codesJohannes Sixt, Jul 4, 2009
  6. 5/7 run_command: encode deadly signal number in the return valueJohannes Sixt, Jul 4, 2009
  7. 6/7 run_command: report failure to execute the program, but optionally don'tJohannes Sixt, Jul 4, 2009
  8. 7/7 receive-pack: remove unnecessary run_status reportJohannes Sixt, Jul 4, 2009
  9. Johannes SixtJul 5, 2009
  10. Junio C HamanoJul 5, 2009
  11. Johannes SixtJul 6, 2009
  12. 1/7 MinGW: truncate exit()'s argument to lowest 8 bitsJohannes Sixt, Jul 5, 2009

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.