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

Re: [PATCH] Redirect stderr to a pipe before redirecting stdout to stderr

From
Johannes Sixt <j.sixt@viscovery.net>
Date
Mar 3, 2008, 08:07 UTC
Message-ID
<47CBB1C7.8010201@viscovery.net>
In-Reply-To
<20080303082111.24d3ea73.chriscool@tuxfamily.org>
Christian Couder schrieb:
Show 12 quoted lines
> With this patch, in the 'start_command' function after forking
> we now take care of stderr in the child process before stdout.
> 
> This way if 'start_command' is called with a 'child_process'
> argument like this:
> 
> 	.err = -1;
> 	.stdout_to_stderr = 1;
> 
> then stderr will be redirected to a pipe before stdout is
> redirected to stderr. So we can now get the process' stdout
> from the pipe (as well as its stderr).

Makes sense. All current users of .stdout_to_stderr don't modify the stderr channel, so this patch should not make a difference.

But please also adjust Documentation/technical/api-run-command.txt because it documents the behavior that you are modifying.

-- Hannes
Previous: Christian Couder
Message 4 of 4 in “Redirect stderr to a pipe before redirecting stdout to stderr”
  1. Redirect stderr to a pipe before redirecting stdout to stderrChristian Couder, Mar 3, 2008
  2. Shawn O. PearceMar 3, 2008
  3. Christian CouderMar 3, 2008
  4. Johannes SixtMar 3, 2008

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.