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

Re: [RFC] sending errors to stdout under $PAGER

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Feb 17, 2008, 19:38 UTC
Message-ID
<alpine.LSU.1.00.0802171937070.30505@racer.site>
In-Reply-To
<7vr6fbzbky.fsf@gitster.siamese.dyndns.org>
Hi,
On Sun, 17 Feb 2008, Junio C Hamano wrote:
Show 21 quoted lines
> Jeff King <peff@peff.net> writes:
> 
> > On Sun, Feb 17, 2008 at 09:56:27AM -0800, Junio C Hamano wrote:
> >
> >> > Wouldn't it be better/safer to redirect stderr to the pager in the 
> >> > first place?
> >> >
> >> > So, instead of the current
> >> >
> >> > 	foo | less
> >> > use
> >> > 	foo 2>&1 | less
> >> 
> >> I like it.  Much nicer.  Thanks.
> >
> > This will also put the stderr of any sub-programs into the pager, 
> > which is probably worse if you have, e.g., a chatty external diff 
> > program. I don't know if we care enough about that.
> 
> We'll soon find out and the change would be a single liner that is very 
> easy to back out, so let's put it in and see what happens ;-).

I think this is a non-problem. External diff programs are virtually all scripts written for git, because of the calling convention. So all this does is tell the people who use external diff wrappers (and let them be chatty) to fix their scripts.

Ciao, Dscho

Previous: Junio C Hamano
Message 9 of 9 in “[RFC] sending errors to stdout under $PAGER”
  1. Junio C HamanoFeb 16, 2008
  2. Shawn O. PearceFeb 17, 2008
  3. Jeff KingFeb 17, 2008
  4. Edgar ToernigFeb 17, 2008
  5. Johannes SchindelinFeb 17, 2008
  6. Junio C HamanoFeb 17, 2008
  7. Jeff KingFeb 17, 2008
  8. Junio C HamanoFeb 17, 2008
  9. Johannes SchindelinFeb 17, 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.