Hi, On Sun, 17 Feb 2008, Junio C Hamano wrote: > Jeff King <peff@xxxxxxxx> 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 - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
