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

Re: [PATCH] notice error exit from pager

From
Clemens Buchacher <drizzd@aon.at>
Date
Aug 1, 2011, 20:35 UTC
Message-ID
<20110801203544.GA22431@toss>
In-Reply-To
<7vd3goj38n.fsf@alter.siamese.dyndns.org>
On Mon, Aug 01, 2011 at 01:17:28PM -0700, Junio C Hamano wrote:
Show 7 quoted lines
> Clemens Buchacher <drizzd@aon.at> writes:
> 
> > Since there is no reliable way to handle error reporting in the
> > parent, produce the output in the child instead.
> 
> Hmm, how does this interact with your earlier "error_routine: use parent's
> stderr if exec fails" patch?

The use case addressed in this patch (i.e., the pager) does not strictly dependent on the previous patch. But Johannes noted that in general, error output of the child could be redirected, hiding the execve failure again. The error_routine patch avoids that by writing to the parent's stderr stream.

Clemens
Previous: Junio C Hamano
Message 7 of 7 in “notice error exit from pager”
  1. notice error exit from pagerClemens Buchacher, Jul 26, 2011
  2. Linus TorvaldsJul 26, 2011
  3. Johannes SixtJul 27, 2011
  4. error_routine: use parent's stderr if exec failsClemens Buchacher, Jul 27, 2011
  5. notice error exit from pagerClemens Buchacher, Aug 1, 2011
  6. Junio C HamanoAug 1, 2011
  7. Clemens BuchacherAug 1, 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.